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

Massage group title padding. Truncate text instead of wrapping it. Add title attribute to options for usability. #1

Merged

Conversation

cjcenizal
Copy link

No description provided.

@nreese nreese merged commit 4e413c8 into nreese:virtualized_combo_box Apr 16, 2018
@nreese
Copy link
Owner

nreese commented Apr 16, 2018

thanks for the help. This looks a lot better

nreese pushed a commit that referenced this pull request Apr 17, 2018
nreese pushed a commit that referenced this pull request Apr 18, 2018
nreese added a commit that referenced this pull request Apr 24, 2018
…#670)

* use react-virtualized to virtualize combo box options list

* use smaller width and height

* include group label in matching options list

* add better text for example description

* dynamically set width and height

* Massage group title padding. Truncate text instead of wrapping it. Add title attribute to options for usability. (#1)

* remove console.log and fix spelling

* fix problems with settig focus on active option

* more keyboard accessiblity work

* Combo box focus state and text overflow (#2)

* Call setState instead of setting activeOptionIndex directly.

* Clear activeOptionIndex when you click the input.

* Prevent a lot of input from overflowing the container.

* Allow disabled options to be focused but not selected.

* add throttle to incrementActiveOptionIndex to avoid keypresses getting UI out of sync

* rowHeight prop

* remove unneeded const

* fix spacing in example text, fix lodash import

* skip disabled options when using keyboard

* Revert "skip disabled options when using keyboard"

This reverts commit 47fa3ef.
@cjcenizal cjcenizal deleted the combo-box-virtualization branch June 8, 2018 22:26
nreese pushed a commit that referenced this pull request Sep 11, 2019
Update defazio fork to 13.0
nreese pushed a commit that referenced this pull request Mar 22, 2020
* removed duplicate icons

* Fixing changelog and updating tests (#1)

Co-authored-by: Caroline Horn <[email protected]>
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