Skip to content
This repository has been 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
@alinais alinais added this to silviuavram in Core Team Jan 21, 2019
@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
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
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
No open projects
Core Team
  
silviuavram
Development

Successfully merging this pull request may close these issues.

None yet

5 participants