Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

proper handling for paste & empty strings #6

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

proper handling for paste & empty strings #6

wants to merge 7 commits into from

Conversation

whyicantusemyemailasusername

Hi,
here the simple patch to support zero-length suggestions - may be useful if you need to show suggestions even if user removed everything from the field; it also add proper handling to text paste events.
Thanks

@RaffaeleSgarro
Copy link
Owner

Thanks for your patch :)

@whyicantusemyemailasusername
Copy link
Author

just to be more specific:

  • there was an issue with transition "field with something in it" -> "empty field" - no notification fired in this case
  • when you ctrl-v something into the field you also didn't get notification

…n component reattach/switch tabs/etc...

change api: showSuggestions(map) instead of addSuggestion(...)
…n component reattach/switch tabs/etc...

change api: showSuggestions(map) instead of addSuggestion(...)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants