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

fix(Dropdown): keep focused style when focus is on List #1258

Merged
merged 7 commits into from
Apr 25, 2019

Conversation

silviuaavram
Copy link
Collaborator

On Single and Multiple selections, when Dropdown was open, focus was moved to the List. After that, the underlined focused style was removed.

This PR aims to fix it.

@codecov
Copy link

codecov bot commented Apr 24, 2019

Codecov Report

Merging #1258 into master will increase coverage by 0.03%.
The diff coverage is 25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1258      +/-   ##
==========================================
+ Coverage    72.1%   72.14%   +0.03%     
==========================================
  Files         731      731              
  Lines        5592     5596       +4     
  Branches     1612     1636      +24     
==========================================
+ Hits         4032     4037       +5     
+ Misses       1555     1554       -1     
  Partials        5        5
Impacted Files Coverage Δ
...ackages/react/src/components/Dropdown/Dropdown.tsx 76.96% <25%> (-0.03%) ⬇️
...ct/src/components/Dropdown/DropdownSearchInput.tsx 53.84% <0%> (+15.38%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c5d373...e44f17d. Read the comment docs.

@silviuaavram silviuaavram added accessibility All the Accessibility tasks and bugs should be tagged with this. 🚀 ready for review and removed 🚧 WIP labels Apr 24, 2019
@silviuaavram silviuaavram merged commit e4822e0 into master Apr 25, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix-focus-style-dropdown branch April 25, 2019 10:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility All the Accessibility tasks and bugs should be tagged with this. 🚀 ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants