From e5807f48aac0bdd7144a33c3743dd400ebbc8977 Mon Sep 17 00:00:00 2001 From: jlukic Date: Tue, 26 May 2015 18:04:55 -0400 Subject: [PATCH] Fix scrolling menu item height --- RELEASE-NOTES.md | 1 + src/themes/default/modules/dropdown.variables | 11 ++++------- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/RELEASE-NOTES.md b/RELEASE-NOTES.md index 401e53ba7..fdacdf41d 100644 --- a/RELEASE-NOTES.md +++ b/RELEASE-NOTES.md @@ -182,6 +182,7 @@ - **Popup** - Fixed bug which could cause pre-existing inline popup to be removed from DOM after hiding - **Popup** - Fixes popup offstage position calculations with pages including horizontal scrollbars - **Reveal** - Removed `masked` reveal, all reveals are masked by default +- **RTL** - Fixed `rtl: 'both'` in `semantic.json` not building both versions of source correctly. - **Search** - Search will no longer incorrectly produce an error when API settings are passed through metadata - **Sidebar** - Top/Bottom sidebar will now show scroll bars correctly when taller than 100% page height - **Sidebar** - Fixed bug where having a `style[title]` in page causing page not to be pushed correctly diff --git a/src/themes/default/modules/dropdown.variables b/src/themes/default/modules/dropdown.variables index 7c27ae4da..d6689f032 100644 --- a/src/themes/default/modules/dropdown.variables +++ b/src/themes/default/modules/dropdown.variables @@ -120,7 +120,7 @@ --------------------*/ /* Selection */ @selectionMinWidth: 200px; -@selectionBorderEmWidth: 0.0785em; /* 2px / em size */ +@selectionBorderEmWidth: 0.0714285em; /* 1px / em size */ @selectionMinHeight: @defaultInputLineHeight + (@selectionVerticalPadding * 2) - @selectionBorderEmWidth; @selectionBackground: @white; @selectionDisplay: inline-block; @@ -305,7 +305,7 @@ --------------------*/ /* Scrolling */ -@scrollingMinWidth: 250px; +@scrollingMinWidth: 15em; @scrollingMenuItemBorder: @selectionItemDivider; @scrollingMobileMaxItems: 4; @@ -313,7 +313,7 @@ @scrollingComputerMaxItems: 8; @scrollingWidescreenMaxItems: 12; -@scrollingBorderEMWidth: 0.0785em; /* 1px / em size */ +@scrollingBorderEMWidth: 0em; /* 0px / em size */ @scrollingItemHeight: (@itemVerticalPadding * 2) + @itemLineHeight + @scrollingBorderEMWidth; @scrollingMobileMaxMenuHeight: (@scrollingItemHeight * @scrollingMobileMaxItems); @scrollingTabletMaxMenuHeight: (@scrollingItemHeight * @scrollingTabletMaxItems); @@ -343,10 +343,7 @@ /* Floating */ @floatingMenuDistance: 0.5em; -@floatingMenuBoxShadow: - 0px 2px 4px 0px @borderColor, - 0px 0px 10px 0px @internalBorderColor !important -; +@floatingMenuBoxShadow: @floatingShadow; @floatingMenuBorderRadius: @borderRadius; /* Pointing */