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

Flex nav #1

Merged
merged 2 commits into from
Mar 9, 2018
Merged

Flex nav #1

merged 2 commits into from
Mar 9, 2018

Conversation

snide
Copy link

@snide snide commented Mar 9, 2018

Sets some reasonable max-width defaults and fixes some issues in mobile / searching.

@cjcenizal cjcenizal merged commit ac1edec into cjcenizal:fixed-side-nav Mar 9, 2018
cjcenizal added a commit that referenced this pull request Apr 16, 2018
cjcenizal pushed a commit that referenced this pull request Apr 28, 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 pushed a commit that referenced this pull request Mar 17, 2020
Update defazio fork to 13.0
cjcenizal pushed a commit that referenced this pull request Mar 17, 2020
* removed duplicate icons

* Fixing changelog and updating tests (#1)

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