Skip to content

Add focus listener to JSpinner text - #13

Merged
hinerm merged 1 commit into
masterfrom
select-jspinner-text
Apr 30, 2015
Merged

Add focus listener to JSpinner text#13
hinerm merged 1 commit into
masterfrom
select-jspinner-text

Conversation

@hinerm

@hinerm hinerm commented Apr 30, 2015

Copy link
Copy Markdown
Member

This (mostly) ensures the text of a JSpinner is selected on focus. On OSX it
seems to always work with tabs, and mostly work with mouse clicking. However
there does seem to be a race condition where the text selection can be lost,
presumably from the JSPinner firing a validation event after the text field's
selection has been processed.

See #12 for future work.

@hinerm

hinerm commented Apr 30, 2015

Copy link
Copy Markdown
Member Author

@ctrueden is this OK..? 🐙

This (mostly) ensures the text of a JSpinner is selected on focus. On OSX it
seems to always work with tabs, and mostly work with mouse clicking. However
there does seem to be a race condition where the text selection can be lost,
presumably from the JSPinner firing a validation event after the text field's
selection has been processed.

See #12 for future work.
@hinerm
hinerm force-pushed the select-jspinner-text branch from 3d3b555 to ff1fa86 Compare April 30, 2015 18:17
@hinerm
hinerm merged commit ff1fa86 into master Apr 30, 2015
@hinerm
hinerm deleted the select-jspinner-text branch April 30, 2015 18:18
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.

1 participant