Skip to content

Commit

Permalink
Fix the heading hierarchy in the site editor (WordPress#51696)
Browse files Browse the repository at this point in the history
  • Loading branch information
youknowriad authored and sethrubenstein committed Jul 13, 2023
1 parent df58591 commit 7da4e62
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ function SidebarNavigationScreenDetailsPanel( { title, children, spacing } ) {
{ title && (
<Heading
className="edit-site-sidebar-navigation-details-screen-panel__heading"
level={ 3 }
level={ 2 }
>
{ title }
</Heading>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ export default function SidebarNavigationScreen( {
<Heading
className="edit-site-sidebar-navigation-screen__title"
color={ 'white' }
level={ 2 }
level={ 1 }
size={ 20 }
>
{ ! isPreviewingTheme()
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/specs/site-editor/command-center.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ test.describe( 'Site editor command center', () => {
.click();
await page.keyboard.type( 'index' );
await page.getByRole( 'option', { name: 'index' } ).click();
await expect( page.getByRole( 'heading', { level: 2 } ) ).toHaveText(
await expect( page.getByRole( 'heading', { level: 1 } ) ).toHaveText(
'Index'
);
} );
Expand Down

0 comments on commit 7da4e62

Please sign in to comment.