Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

New param for config and clear last item #115

Merged
merged 3 commits into from
May 29, 2019

Conversation

TchernyavskyDaniil
Copy link
Contributor

Hi @Mobius1 , thx for your Selectr. I have a case like that, Mobile non-active

And it's ok, but https://github.com/Mobius1/Selectr/blob/master/src/selectr.js#L1073

So, I added a new param for config.

And I have a question about these two lines:
https://github.com/Mobius1/Selectr/blob/master/src/selectr.js#L1184
https://github.com/Mobius1/Selectr/blob/master/src/selectr.js#L1188

What is that? I have a bug on mobile devices with these settings when I use this.selector.mobileDevice = false;, but nvm, maybe it's my fail.

And about second, I added param isLastKey for clear method. In my case, I needed clear one item after keycode backspace, so, I can't find a method for that or I missed?

@Mobius1 Mobius1 merged commit 6fb629c into Mobius1:master May 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants