Skip to content
This repository was 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
silviuaavram merged 7 commits intomasterfrom
fix-focus-style-dropdown
Apr 25, 2019
Merged

fix(Dropdown): keep focused style when focus is on List#1258
silviuaavram merged 7 commits intomasterfrom
fix-focus-style-dropdown

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 force-pushed the fix-focus-style-dropdown branch from e4401b9 to c2d3dc0 Compare April 24, 2019 10:47
@silviuaavram silviuaavram force-pushed the fix-focus-style-dropdown branch from 854462c to 16f5384 Compare April 24, 2019 14:15
@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.

2 participants