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

A bugfix for the repeated focusing issue #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

erhead
Copy link

@erhead erhead commented Jan 18, 2019

Currently there's an issue in behavior which allows to enter an irrelevant value into the input. The steps to reproduce are as follows:

  • Start typing an irrelevant value
  • Click on that value (it must become selected after that)
  • Click elsewhere to make the input lose focus.
    After that an irrelevant value stays in the input.
    The reason is that when we click on the input, the focus handler is called and therefore the "abbreviationBeforeFocus" variable is initialized with new irrelevant value from the input.
    The bugfix is to prevent the component from repeated focusing.

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.

None yet

1 participant