From d4dce9982ef740101924a3dea8eee003688625f8 Mon Sep 17 00:00:00 2001 From: George Mamadashvili Date: Thu, 18 Jun 2026 11:49:55 +0400 Subject: [PATCH 1/2] Site Editor: Introduce `isHidden` prop for `SidebarNavigationItem` --- .../sidebar-navigation-item/index.js | 11 ++ .../sidebar-navigation-screen-main/index.js | 110 +++++++++--------- 2 files changed, 65 insertions(+), 56 deletions(-) diff --git a/packages/edit-site/src/components/sidebar-navigation-item/index.js b/packages/edit-site/src/components/sidebar-navigation-item/index.js index 687f7f81b1a3ee..d504cb6f364211 100644 --- a/packages/edit-site/src/components/sidebar-navigation-item/index.js +++ b/packages/edit-site/src/components/sidebar-navigation-item/index.js @@ -22,6 +22,17 @@ import { SidebarNavigationContext } from '../sidebar'; const { useHistory, useLink, useLocation } = unlock( routerPrivateApis ); export default function SidebarNavigationItem( { + isHidden = false, + ...props +} ) { + if ( isHidden ) { + return null; + } + + return ; +} + +function SidebarNavigationItemContent( { className, icon, withChevron = false, diff --git a/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js b/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js index 666947b69cc73b..8dbf3bfe631e05 100644 --- a/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js +++ b/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js @@ -23,52 +23,51 @@ import SidebarNavigationItem from '../sidebar-navigation-item'; export function MainSidebarNavigationContent( { isBlockBasedTheme = true } ) { return ( - { isBlockBasedTheme && ( - <> - - { _x( 'Identity', 'site identity' ) } - - - { __( 'Styles' ) } - - - { __( 'Pages' ) } - - - { __( 'Navigation' ) } - - - ) } - { ! isBlockBasedTheme && ( - - { __( 'Styles' ) } - - ) } + + { _x( 'Identity', 'site identity' ) } + + + { __( 'Styles' ) } + + + { __( 'Pages' ) } + + + { __( 'Navigation' ) } + + + { __( 'Styles' ) } + { __( 'Patterns' ) } - { isBlockBasedTheme && ( - - { __( 'Templates' ) } - - ) } + + { __( 'Templates' ) } + ); } From ed41e5ae4e5377fb25ebbe87904d02e7cfdcf0b6 Mon Sep 17 00:00:00 2001 From: George Mamadashvili Date: Fri, 19 Jun 2026 17:01:05 +0400 Subject: [PATCH 2/2] Avoid Styles item duplication --- .../sidebar-navigation-screen-main/index.js | 21 +++++++------------ 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js b/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js index 8dbf3bfe631e05..950adb8795c8f1 100644 --- a/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js +++ b/packages/edit-site/src/components/sidebar-navigation-screen-main/index.js @@ -33,11 +33,15 @@ export function MainSidebarNavigationContent( { isBlockBasedTheme = true } ) { { _x( 'Identity', 'site identity' ) } { __( 'Styles' ) } @@ -59,15 +63,6 @@ export function MainSidebarNavigationContent( { isBlockBasedTheme = true } ) { > { __( 'Navigation' ) } - - { __( 'Styles' ) } -