From e1499e61d6ffe9cf59b6b4336be4056f56ad8b13 Mon Sep 17 00:00:00 2001
From: jlukic <jacklukic@gmail.com>
Date: Wed, 18 Mar 2015 17:57:06 -0400
Subject: [PATCH] Fix popup in menu, add min-height

---
 src/definitions/collections/menu.less         | 9 +++++----
 src/themes/default/collections/menu.variables | 1 +
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/definitions/collections/menu.less b/src/definitions/collections/menu.less
index c1e881406..98adcbe6e 100755
--- a/src/definitions/collections/menu.less
+++ b/src/definitions/collections/menu.less
@@ -35,6 +35,7 @@
   font-weight: @fontWeight;
   box-shadow: @boxShadow;
   border-radius: @borderRadius;
+  min-height: @minHeight;
 }
 
 .ui.menu:after {
@@ -203,10 +204,10 @@
      Popup
 ---------------*/
 
-.ui.menu .popup {
+.ui.menu .ui.popup {
   display: none;
 }
-.ui.menu .visible.popup {
+.ui.menu .ui.visible.popup {
   display: block;
 }
 
@@ -813,6 +814,7 @@ Floated Menu / Item
   box-shadow: none;
   background: @secondaryActiveItemBackground;
   color: @secondaryActiveItemColor;
+  border-radius: @secondaryItemBorderRadius;
 }
 
 /* Active Hover */
@@ -1691,8 +1693,7 @@ Floated Menu / Item
 .ui.menu.attached {
   margin: 0rem;
   border-radius: 0px;
-  /* avoid rgba multiplying */
-  box-shadow: @attachedBoxShadow;
+  box-shadow: @attachedBoxShadow; /* avoid rgba multiplying */
 }
 .ui.top.attached.menu {
   border-radius: @borderRadius @borderRadius 0em 0em;
diff --git a/src/themes/default/collections/menu.variables b/src/themes/default/collections/menu.variables
index 1c3c582d8..729729f7f 100644
--- a/src/themes/default/collections/menu.variables
+++ b/src/themes/default/collections/menu.variables
@@ -27,6 +27,7 @@
   @subtleShadow
 ;
 @borderRadius: @defaultBorderRadius;
+@minHeight: (@itemVerticalPadding * 2) + 1em;
 
 /* Menu Item */
 @itemVerticalPadding: @relativeSmall;