fix(menu): Only show scrollbar when menu item is too big

RobJacobs · lynnmercier · commit fe7d4c80bc03 · 2017-09-22T10:12:51.000-07:00
Also fixes select's menu Resolves #1247
diff --git a/packages/mdc-menu/simple/mdc-simple-menu.scss b/packages/mdc-menu/simple/mdc-simple-menu.scss
@@ -37,7 +37,8 @@ $mdc-simple-menu-item-fade-duration: .3s;
   background-color: white;
   white-space: nowrap;
   opacity: 0;
-  overflow: hidden;
+  overflow-x: hidden;
+  overflow-y: auto;
   box-sizing: border-box;
   will-change: transform, opacity;
   z-index: 4;
diff --git a/packages/mdc-select/mdc-select.scss b/packages/mdc-select/mdc-select.scss
@@ -85,7 +85,6 @@
     left: 0;
     max-height: 100%;
     transform-origin: center center;
-    overflow-y: scroll;
     z-index: 4; // Should pop up above everything else. temporary-drawer is next highest at 3.
   }
 

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,6 @@`
`85`	`85`	`left: 0;`
`86`	`86`	`max-height: 100%;`
`87`	`87`	`transform-origin: center center;`
`88`		`- overflow-y: scroll;`
`89`	`88`	`z-index: 4; // Should pop up above everything else. temporary-drawer is next highest at 3.`
`90`	`89`	`}`
`91`	`90`