feat(nav): updated css vars

patternfly · Nov 10, 2021 · 12a2ce5 · 12a2ce5
1 parent 9e3e776
commit 12a2ce5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/src/patternfly/components/Menu/menu.scss b/src/patternfly/components/Menu/menu.scss
@@ -187,7 +187,6 @@
   z-index: var(--pf-c-menu--ZIndex);
   width: var(--pf-c-menu--Width);
   min-width: var(--pf-c-menu--MinWidth);
-  overflow: visible;
   background-color: var(--pf-c-menu--BackgroundColor);
   box-shadow: var(--pf-c-menu--BoxShadow);
 
@@ -203,7 +202,7 @@
   }
 
   // nested flyout menu
-  &.pf-m-flyout,
+  & &.pf-m-flyout,
   &.pf-m-flyout & {
     position: absolute;
     top: var(--pf-c-menu--m-flyout__menu--Top);

diff --git a/src/patternfly/components/Nav/nav.scss b/src/patternfly/components/Nav/nav.scss
@@ -350,22 +350,16 @@
   // Menu item description
   --pf-c-nav--c-menu__item-description--Color: var(--pf-global--Color--400);
 
-  // stylelint-disable length-zero-no-unit
-  // Needs a unit because of type checking for use in calc()
-  --pf-c-nav--c-menu--m-flyout--top-offset: 0px;
+  // Menu flyout
   --pf-c-nav--c-menu--m-flyout--left-offset: #{pf-size-prem(4px)};
-  --pf-c-nav--c-menu--m-flyout--m-top--bottom-offset: 0px;
   --pf-c-nav--c-menu--m-flyout--m-left--right-offset: #{pf-size-prem(4px)};
-  // stylelint-enable
-
-  // Menu flyout
-  --pf-c-nav--c-menu--m-flyout--Top: var(--pf-c-nav--c-menu--m-flyout--top-offset);
+  --pf-c-nav--c-menu--m-flyout--Top: 0;
   --pf-c-nav--c-menu--m-flyout--c-menu--Top: calc(var(--pf-c-nav--c-menu--m-flyout--Top) * -1);
   --pf-c-nav--c-menu--m-flyout--c-menu--Right: auto;
   --pf-c-nav--c-menu--m-flyout--c-menu--Bottom: auto;
   --pf-c-nav--c-menu--m-flyout--c-menu--Left: calc(100% - var(--pf-c-nav--c-menu--m-flyout--left-offset));
   --pf-c-nav--c-menu--m-flyout--m-left--Right: calc(100% - var(--pf-c-nav--c-menu--m-flyout--m-left--right-offset));
-  --pf-c-nav--c-menu--m-flyout--m-top--Bottom: var(--pf-c-nav--c-menu--m-flyout--m-top--bottom-offset);
+  --pf-c-nav--c-menu--m-flyout--m-top--Bottom: 0;
   --pf-c-nav--c-menu--m-flyout--BoxShadow: var(--pf-global--BoxShadow--lg);
 
   // Menu flyout item