Skip to content

Fix constrained tabbing failures with Safari and Firefox #47426

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
afercia merged 11 commits into from
Feb 14, 2023
Merged

Fix constrained tabbing failures with Safari and Firefox #47426

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
7c087b4
Make the modal content container focusable when scrollable.
afercia Jan 20, 2023
80ea82d
Add test, improve existing a11y tests and run them with all browsers.
afercia Jan 20, 2023
7a87f07
Fix useConstrainedTabbing edge cases.
afercia Jan 23, 2023
66a4693
Add basic focus style for the scrollable section.
afercia Jan 24, 2023
2ce7d2a
Update snapshots.
afercia Jan 25, 2023
4fcb5b7
Clean up a11y test.
afercia Jan 25, 2023
3900724
Refine test.
afercia Jan 25, 2023
abc31c2
Remove scrollableContentLabel prop.
afercia Feb 9, 2023
04b9c37
Restore conditional aria-label for the scrollable section.
afercia Feb 9, 2023
47a5430
Update test after change to closeButtonLabel default.
afercia Feb 9, 2023
50da893
Add comment to clarify test.
afercia Feb 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Refine test.
  • Loading branch information
@afercia
afercia committed Feb 9, 2023
commit 390072452c8e88b16b83ebd8d27234673fc579b9
7 changes: 4 additions & 3 deletions test/e2e/specs/editor/various/a11y.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,12 +170,13 @@ test.describe( 'a11y (@firefox, @webkit)', () => {
await page.keyboard.press( 'Shift+Tab' );
await expect( preferencesModalContent ).toBeFocused();

// Make the Blocks tab panel content shorter by searching for a block.
// It's now not scrollable. Check it's not focusable.
// Make the Blocks tab panel content shorter by searching for a block
// that doesn't exist. The content only shows 'No blocks found' and it's
// not scrollable any longer. Check it's not focusable.
await clickAndFocusTab( blocksTab );
await page.type(
'role=searchbox[name="Search for a block"i]',
'cover'
'qwerty'
);
await clickAndFocusTab( blocksTab );
await page.keyboard.press( 'Shift+Tab' );
Expand Down