Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Optional autofocus on typeahead selection #2270

Closed
wants to merge 4 commits into from

Conversation

angi-
Copy link

@angi- angi- commented May 28, 2014

There are some issues when you want an input to clear value on focus and after selection it gets cleared again because of the autofocus. I added an optional parameter for autofocus on selection, it is disabled by default.

I added an optional parameter for autofocus on selection
@pkozlowski-opensource
Copy link
Member

@angi-
Copy link
Author

angi- commented May 28, 2014

Sorry for that, now it passed. Do you think autofocus should be enabled by default? This way it won't affect previous users but others can disable it.

@angi- angi- changed the title Optional autofocus on selection Optional autofocus on typeahead selection May 29, 2014
@angi-
Copy link
Author

angi- commented May 29, 2014

Now the optional parameter disables the autofocus leaving the old functionality intact.

This also adresses the issue #1903

@wesleycho wesleycho added this to the 0.13.1 milestone Apr 5, 2015
@wesleycho
Copy link
Contributor

Can you squash your commits and rebase off of current master?

Scheduling this for 0.13.1, this just needs documentation updates.

@angi-
Copy link
Author

angi- commented Apr 14, 2015

I have done this on another pull request, I also created automated tests: #3211

Let me know if I can further help to get this update in the next release.

@karianna
Copy link
Contributor

@angi- So should we close this PR?

@wesleycho
Copy link
Contributor

This just needs documentation updates and it should be good to go.

@wesleycho wesleycho modified the milestones: 0.13.1 (npm), 0.13.2 (Performance) Jul 23, 2015
@wesleycho wesleycho modified the milestones: 0.13.2 (1.4 support), 0.13.3 (Performance) Aug 2, 2015
@wesleycho wesleycho modified the milestones: 0.13.3 (Fixes), 0.13.4 (Performance) Aug 9, 2015
@wesleycho
Copy link
Contributor

Closing, as it has been superceded by #4212.

@wesleycho wesleycho closed this Aug 16, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants