Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

fix(Dropdown): Highlights selected option in single selection #726

Merged
merged 9 commits into from
Jan 29, 2019

Conversation

silviuaavram
Copy link
Collaborator

@silviuaavram silviuaavram commented Jan 15, 2019

Highlights selected option when opening dropdown.

Based on downshift-js/downshift#652 and since now it's released, this PR will be able to perform the necessary fix.

@silviuaavram silviuaavram self-assigned this Jan 15, 2019
@silviuaavram silviuaavram force-pushed the fix/highlight-on-selected branch from e07fe1d to 461ea53 Compare January 22, 2019 16:04
@silviuaavram silviuaavram changed the title Highlights selected option when opening dropdown Highlights defaultHighlightedIndex when opening dropdown Jan 25, 2019
@silviuaavram silviuaavram changed the title Highlights defaultHighlightedIndex when opening dropdown feat(Dropdown): Highlights selected option in single selection Jan 25, 2019
@silviuaavram silviuaavram changed the title feat(Dropdown): Highlights selected option in single selection fix(Dropdown): Highlights selected option in single selection Jan 25, 2019
@silviuaavram silviuaavram force-pushed the fix/highlight-on-selected branch from 461ea53 to 183261d Compare January 25, 2019 09:47
Copy link
Member

@layershifter layershifter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Downshift has highlightedIndex, too. Can we implement it as autocontrolled prop?

@silviuaavram silviuaavram force-pushed the fix/highlight-on-selected branch from 8d156d8 to ee9100f Compare January 28, 2019 20:40
@silviuaavram
Copy link
Collaborator Author

@layershifter let's create a new issue with making those props autocontrolled. It will be a bit more complicated, as our default is actually initial in Downshift, while default there is the value that prop will be used at every change (selection, close/open).

@silviuaavram silviuaavram merged commit e2fe1e3 into master Jan 29, 2019
@layershifter layershifter deleted the fix/highlight-on-selected branch January 29, 2019 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants