Skip to content

Commit

Permalink
Address feedback.
Browse files Browse the repository at this point in the history
  • Loading branch information
jasmussen committed Aug 25, 2021
1 parent a646b05 commit 8994a78
Showing 1 changed file with 2 additions and 24 deletions.
26 changes: 2 additions & 24 deletions packages/block-library/src/navigation/style.scss
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
// Navigation block and menu item styles.
// as that block is meant to behave as menu items when leveraged.
// These styles also affect the Page List block when used inside your navigation block.
//
// Classes:
Expand Down Expand Up @@ -80,7 +79,7 @@
}

// Submenu indicator.
.wp-block-navigation-link__submenu-icon {
.wp-block-navigation__submenu-icon {
align-self: center;
height: inherit;
line-height: 0;
Expand Down Expand Up @@ -181,25 +180,6 @@
visibility: visible;
opacity: 1;
}

// Page list menu items.
&:hover {
cursor: pointer;

> .submenu-container {
visibility: visible;
opacity: 1;
}
}

&:focus-within {
cursor: pointer;

> .submenu-container {
visibility: visible;
opacity: 1;
}
}
}

// Submenu indentation when there's a background.
Expand Down Expand Up @@ -304,9 +284,7 @@

// Default background and font color.
.wp-block-navigation:not(.has-background) {
// @todo: check this one separately.
.submenu-container, // This target items created by the Page List block.
.wp-block-navigation__container .wp-block-navigation__submenu-container {
.wp-block-navigation__submenu-container {
// Set a background color for submenus so that they're not transparent.
// NOTE TO DEVS - if refactoring this code, please double-check that
// submenus have a default background color, this feature has regressed
Expand Down

0 comments on commit 8994a78

Please sign in to comment.