Skip to content

Comments

Combo box focus state and text overflow#2

Merged
nreese merged 4 commits intonreese:virtualized_combo_boxfrom
cjcenizal:combo-box-focus-state
Apr 17, 2018
Merged

Combo box focus state and text overflow#2
nreese merged 4 commits intonreese:virtualized_combo_boxfrom
cjcenizal:combo-box-focus-state

Conversation

@cjcenizal
Copy link

No description provided.

@cjcenizal cjcenizal force-pushed the combo-box-focus-state branch from 0fa364b to eb49113 Compare April 16, 2018 23:47
@nreese nreese merged commit d7172a5 into nreese:virtualized_combo_box Apr 17, 2018
nreese pushed a commit that referenced this pull request Apr 17, 2018
* 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.
nreese pushed a commit that referenced this pull request Apr 18, 2018
* 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.
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-focus-state branch June 8, 2018 22:27
nreese pushed a commit that referenced this pull request Sep 11, 2019
nreese pushed a commit that referenced this pull request Mar 22, 2020
* fixed errors

* updated changelog

* Update CHANGELOG.md

* added preview in docs-section

* Update CHANGELOG.md

* added example file

* Fixing anchor and disabled styles, updating example (#2)

Thanks @cchaos

* added tests

* Update CHANGELOG.md

Co-Authored-By: Caroline Horn <549577+cchaos@users.noreply.github.com>

Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>
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