Skip to content
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.

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

Implement Tabs in editor settings #55360

Merged
merged 23 commits into from
Dec 12, 2023
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
ad8819d
implement Tabs in editor settings sidebar
chad1008 Nov 1, 2023
4cb5c69
remove duplicated styles from editor sidebar
chad1008 Nov 1, 2023
ddd8d7f
incorporate initial feedback
chad1008 Nov 3, 2023
4142f5a
pass props to Tabs directly
chad1008 Nov 9, 2023
b67019b
add `closeGeneralSidebar` to `onSelect` callback
chad1008 Nov 16, 2023
b9723d7
set TabPanels to `focuasable={false}`
chad1008 Nov 16, 2023
1fbbef4
detect when sidebar is closed. pass `selectedTabId` of `null`
chad1008 Nov 16, 2023
f3fea72
improve `Tabs` `onSelect` callback
chad1008 Nov 16, 2023
a01b24c
remove `aria-label` and `data-label` props
chad1008 Nov 17, 2023
2e73939
add note explaining null selected tab when sidebar is closed
chad1008 Nov 17, 2023
ed1c930
update e2e test
chad1008 Nov 20, 2023
4d47b34
style updates
chad1008 Nov 20, 2023
1e471ee
update internal component structure to avoid rerenders
chad1008 Nov 22, 2023
9ea4a0e
remove fragment
chad1008 Nov 22, 2023
8e5ad95
prevent infinite loop when opening third party sidebar
chad1008 Nov 24, 2023
e57f8a1
update e2e tests for `Tabs` compatibility
chad1008 Nov 27, 2023
8a7b8f1
fix double top margin on tabpanels
chad1008 Nov 28, 2023
c3eb43e
update to use new tabId prop
chad1008 Dec 8, 2023
73db1df
remove import that is no longer needed after rebase
chad1008 Dec 8, 2023
7fe4d02
fix keyboard navigable blocks test
chad1008 Dec 8, 2023
4d0c547
fix visibility and tab order tests
chad1008 Dec 8, 2023
10507a2
fix footnotes tests
chad1008 Dec 8, 2023
5d20c8b
fix change detection test
chad1008 Dec 9, 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
fix footnotes tests
  • Loading branch information
chad1008 committed Dec 8, 2023
commit 10507a2427396b1bb6641affad2020807806a1d4
4 changes: 2 additions & 2 deletions test/e2e/specs/editor/various/footnotes.spec.js
Original file line number Diff line number Diff line change
@@ -362,7 +362,7 @@ test.describe( 'Footnotes', () => {
await editor.openDocumentSettingsSidebar();
await page
.getByRole( 'region', { name: 'Editor settings' } )
.getByRole( 'button', { name: 'Post' } )
.getByRole( 'tab', { name: 'Post' } )
.click();
await page.locator( 'a:text("2 Revisions")' ).click();
await page.locator( '.revisions-controls .ui-slider-handle' ).focus();
@@ -440,7 +440,7 @@ test.describe( 'Footnotes', () => {
await editor.openDocumentSettingsSidebar();
await page
.getByRole( 'region', { name: 'Editor settings' } )
.getByRole( 'button', { name: 'Post' } )
.getByRole( 'tab', { name: 'Post' } )
.click();

// Visit the published post.
Loading