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

fix(FocusZone): Elements with contenteditable attribute should be respected on Home/End key press #1749

Merged
merged 6 commits into from
Aug 5, 2019

Conversation

sophieH29
Copy link
Contributor

Fixes #1608

When pressing Home \ End keys, focus should not move out of element with contenteditable attribute.

@sophieH29 sophieH29 added 🚀 ready for review 🧰 fix Introduces fix for broken behavior. accessibility All the Accessibility tasks and bugs should be tagged with this. labels Aug 1, 2019
@sophieH29 sophieH29 self-assigned this Aug 1, 2019
@vercel vercel bot temporarily deployed to staging August 1, 2019 16:45 Inactive
@codecov
Copy link

codecov bot commented Aug 1, 2019

Codecov Report

Merging #1749 into master will increase coverage by 0.05%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1749      +/-   ##
=========================================
+ Coverage   69.94%     70%   +0.05%     
=========================================
  Files         868     868              
  Lines        7347    7351       +4     
  Branches     2141    2121      -20     
=========================================
+ Hits         5139    5146       +7     
+ Misses       2202    2199       -3     
  Partials        6       6
Impacted Files Coverage Δ
...eact/src/lib/accessibility/FocusZone/FocusZone.tsx 88.53% <75%> (+0.74%) ⬆️

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 1a662ce...9617a1f. Read the comment docs.

@lucivpav
Copy link
Contributor

lucivpav commented Aug 2, 2019

Is it possible to add test(s) for this?

@sophieH29 sophieH29 merged commit a5140f6 into master Aug 5, 2019
@sophieH29 sophieH29 deleted the fix/fz-contenteditable-info branch August 5, 2019 13:01
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. 🧰 fix Introduces fix for broken behavior. 🚀 ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Home/End don't work in contenteditable divs underneath FocusZone
3 participants