PR #1493 fix(#1478) Button with `display: none` breaks keyboard navigation in <FocusScope contain>

fix(#1478): Button with `display: none` breaks keyboard navigation in <FocusScope contain>

Michael Jordan committed 4 years ago

fix(#1478): refactor to use getFocusableTreeWalker

Michael Jordan committed 4 years ago

fix(#1478): refactor focusPrevious to use getFocusableTreeWalker

Michael Jordan committed 4 years ago

fix(#1478): remove unused focusManager instance

Michael Jordan committed 4 years ago

fix(#1478): refactor useFocusContainment to use getFocusableTreeWalker

Michael Jordan committed 4 years ago

fix(#1478): make syntax more consistent

Michael Jordan committed 4 years ago

fix(#1478): don't tree walk to manage focus when nodeToRestore === document.body

Michael Jordan committed 4 years ago

fix(#1478): Add unit tests for FocusScope edge cases

Michael Jordan committed 4 years ago

fix(#1478): Refactor focusPrevious and tests to remove additional tree walking

Michael Jordan committed 4 years ago

fix(#1570): FocusScope causing unintended focus trap

Michael Jordan committed 4 years ago

fix(#1570): refactor FocusScope causing unintended focus trap

Michael Jordan committed 4 years ago

Merge branch 'main' into Issue-1478

GitHub committed 4 years ago

fix(#1478): Add license notice for adapted code

Michael Jordan committed 4 years ago

Merge branch 'main' into Issue-1478

GitHub committed 4 years ago

fix(#1478): refactor to use userEvent.tab() in tests

Michael Jordan committed 4 years ago

fix(#1478): fix lint errors caused by missing semicolons

Michael Jordan committed 4 years ago

fix(#1478): fix lint errors caused by extra semicolons

Michael Jordan committed 4 years ago

Merge branch 'main' into Issue-1478

GitHub committed 4 years ago

Merge remote-tracking branch 'adobe/main' into Issue-1478

Michael Jordan committed 4 years ago

fix(#1478): FocusScope: refactor getScopeRoot

Michael Jordan committed 4 years ago

Merge branch 'main' into Issue-1478

GitHub committed 4 years ago

fix(#1478): reduce the number of calls to isElementInScope

Michael Jordan committed 4 years ago

fix(#1478): updates per code review

Michael Jordan committed 4 years ago

react-spectrum fix(#1478) Button with `display: none` breaks keyboard navigation in <FocusScope contain> #1493 Merged