From fe7d4c80bc03772e1b0486dd890c4494340c7dcc Mon Sep 17 00:00:00 2001 From: Rob Jacobs Date: Fri, 22 Sep 2017 13:12:51 -0400 Subject: [PATCH] fix(menu): Only show scrollbar when menu item is too big Also fixes select's menu Resolves #1247 --- packages/mdc-menu/simple/mdc-simple-menu.scss | 3 ++- packages/mdc-select/mdc-select.scss | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/mdc-menu/simple/mdc-simple-menu.scss b/packages/mdc-menu/simple/mdc-simple-menu.scss index 511e1cfb134..81283dd8679 100644 --- 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 index ced5b94222a..e0557834276 100644 --- 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. }