fix: move max-width: 100px from menu triggers to icon triggers (#849)

* US246496 removed max-width: 100px from triggers

* Code revision, added changelog

Co-authored-by: [ Cassondra ] <[email protected]>

Author: chrisdo1

CHANGELOG-prerelease.md

@@ -4,8 +4,8 @@
 - [2c950b0](https://github.com/patternfly/patternfly-elements/commit/2c950b08f7638787df50aa5ee6738f1205ea3a9d) fix: Add clearfix within tab and accordion panels
 - [96f0a1b](https://github.com/patternfly/patternfly-elements/commit/96f0a1bdf9c758650e02b20a63ee3fb2fcf11bc9) feat: Add border to the card component
 - [f392a0f](https://github.com/patternfly/patternfly-elements/commit/f392a0f0eeac3b0379794eff4a1b2946e10e883a) fix: Generator needs to convert slot-name to camelCase in js file if dash exists
-- [](https://github.com/patternfly/patternfly-elements/commit/) fix: adjust arrow spacing & alignment on pfe-cta
-
+- [6eed4c3](https://github.com/patternfly/patternfly-elements/commit/6eed4c3dae562e20c12879b2790cb68031e8975e) fix: adjust arrow spacing & alignment on pfe-cta
+- [](https://github.com/patternfly/patternfly-elements/commit/) fix: Move max-width:100px to icon triggers
 
 ## Prerelese 45 ( 2020-04-27 )
 

elements/pfe-navigation/src/_shared-assets.scss

@@ -64,18 +64,11 @@ $variables: (
     margin: 0 !important;
     max-width: 100%; // added
     text-align: center;
-    // max-width: #{pfe-local($cssvar: MaxWidth, $region: trigger, $fallback: 100%)};
 
     @media screen and (min-width: pfe-breakpoint(xs) ) {
-        // --pfe-navigation__trigger--MaxWidth: 100%; // added
         max-width: 100%;
     }
-    @media screen and (min-width: pfe-breakpoint(sm-desktop, $map: false) ) {
-        // --pfe-navigation__trigger--MaxWidth: 100px; // added
-        max-width: 100px;
-    }
     @media screen and (min-width: pfe-breakpoint(xl) ) {
-        // --pfe-navigation__trigger--MaxWidth: 190px; // added
         max-width: 190px;
     }
 }

elements/pfe-navigation/src/pfe-navigation-item.scss

@@ -262,14 +262,11 @@ $variables: map-merge($variables, $nav-item-variables);
 }
 
 // Typography styles need specificity over bootstrap
-
 ::slotted([slot="trigger"]) {
   @extend %trigger;
-}
-
-:host([pfe-icon]) {
-  ::slotted([slot="trigger"]) {
+  :host([pfe-icon]) & {
     margin: 0.25em 0 0 0 !important;
+    max-width: 100px;
   }
 }