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

feat(accessibility): Add FocusZone for ButtonGroup, TabList and Toolbar behaviors #254

Merged
merged 10 commits into from
Oct 3, 2018

Conversation

jurokapsiar
Copy link
Contributor

This PR adds FocusZone handling for Button Group, TabList and Toolbar behaviors.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #254 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #254      +/-   ##
==========================================
+ Coverage   89.62%   89.63%   +<.01%     
==========================================
  Files          62       62              
  Lines        1166     1167       +1     
  Branches      173      173              
==========================================
+ Hits         1045     1046       +1     
  Misses        119      119              
  Partials        2        2
Impacted Files Coverage Δ
src/components/Button/ButtonGroup.tsx 97.29% <100%> (+0.07%) ⬆️

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 550dada...7f3612d. Read the comment docs.

@jurokapsiar jurokapsiar changed the title Add FocusZone for ButtonGroup, TabList and Toolbar behaviors feat/Add FocusZone for ButtonGroup, TabList and Toolbar behaviors Sep 20, 2018
@jurokapsiar jurokapsiar changed the title feat/Add FocusZone for ButtonGroup, TabList and Toolbar behaviors feat(accessibility): Add FocusZone for ButtonGroup, TabList and Toolbar behaviors Sep 20, 2018
@jurokapsiar jurokapsiar closed this Oct 3, 2018
@jurokapsiar jurokapsiar reopened this Oct 3, 2018
@jurokapsiar jurokapsiar merged commit 712c72e into master Oct 3, 2018
@levithomason levithomason deleted the feat/focuszone-behaviors branch October 29, 2019 22:31
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.

4 participants