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

Already on GitHub? Sign in to your account

Add `shouldFocusInput` option [Fix #1541] #2140

Merged
merged 1 commit into from Mar 7, 2014

Conversation

Projects
None yet
2 participants
Owner

kevin-brown commented Feb 22, 2014

This adds a shouldFocusInput option that determines whether or not the focusser should be automatically focused. By default, the focusser will always be focused, unless minimumResultsForSearch is less than zero.

close on single selects took an option params argument which at one point was used to tell the focusser to not automatically focus. This was added at one point when tweaking the mask and later reverted to fix a bug that came as a result. The leftovers of this code has been removed in this commit.

@kevin-brown kevin-brown Add `shouldFocusInput` option [Fix #1541]
This adds a `shouldFocusInput` option that determines whether or
not the focusser should be automatically focused.  By default, the
focusser will always be focused, unless `minimumResultsForSearch`
is less than zero.

`close` on single selects took an option `params` argument which
at one point was used to tell the focusser to not automatically
focus.  This was added at one point when tweaking the mask [1]
and later reverted to fix a bug that came as a result [2].  The
leftovers of this code has been removed in this commit.

[1]: e162a48
[2]: 9bc68f0
d87e93d

@kevin-brown kevin-brown added a commit that referenced this pull request Mar 7, 2014

@kevin-brown kevin-brown Merge pull request #2140 from ivaynberg/issue_1541
Add `shouldFocusInput` option [Fix #1541]
2d906ee

@kevin-brown kevin-brown merged commit 2d906ee into master Mar 7, 2014

@kevin-brown kevin-brown deleted the issue_1541 branch Mar 7, 2014

ksikka commented Apr 19, 2015

Is the shouldFocusInput option meant to be used by the end user? Saw that it was not documented, wondering if that was intentional or not.

@v-evfila v-evfila pushed a commit to v-evfila/select2 that referenced this pull request Mar 31, 2017

@kevin-brown kevin-brown Merge pull request #2140 from ivaynberg/issue_1541
Add `shouldFocusInput` option [Fix #1541]
454307d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment