From dd45a3c724485de422fe600155dea7f08be10db9 Mon Sep 17 00:00:00 2001 From: Tom Date: Tue, 14 Jan 2014 00:47:05 +0200 Subject: [PATCH 1/3] Prevent menu button color inheritance. Buttons within .ui.menu, that are styled above anchor (`a`) tag, are colored inappropriately. This fixes it. --- src/collections/menu.less | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/collections/menu.less b/src/collections/menu.less index 8a8e1423d..59f8e8bba 100755 --- a/src/collections/menu.less +++ b/src/collections/menu.less @@ -106,19 +106,19 @@ /* Text Color */ .ui.menu .item, -.ui.menu .item > a { +.ui.menu .item > a:not(.button) { color: rgba(0, 0, 0, 0.75); } .ui.menu .item .item, -.ui.menu .item .item > a { +.ui.menu .item .item > a:not(.button) { color: rgba(30, 30, 30, 0.7); } .ui.menu .item .item .item, -.ui.menu .item .item .item > a { +.ui.menu .item .item .item > a:not(.button) { color: rgba(30, 30, 30, 0.6); } .ui.menu .dropdown.item .menu .item, -.ui.menu .dropdown.item .menu .item a { +.ui.menu .dropdown.item .menu .item a:not(.button) { color: rgba(0, 0, 0, 0.75); } /* Hover */ @@ -128,12 +128,12 @@ .ui.menu .item .menu .link.item.hover { color: rgba(0, 0, 0, 0.85); } -.ui.menu .dropdown.item .menu .item a:hover { +.ui.menu .dropdown.item .menu .item a:not(.button):hover { color: rgba(0, 0, 0, 0.85); } /* Active */ .ui.menu .active.item, -.ui.menu .active.item a { +.ui.menu .active.item a:not(.button) { color: rgba(0, 0, 0, 0.85); -moz-border-radius: 0px; @@ -1808,4 +1808,4 @@ } .ui.large.vertical.menu { width: 18rem; -} \ No newline at end of file +} From 8eb03ca4ab32163b7653f8f8c528d1285b0ae6c9 Mon Sep 17 00:00:00 2001 From: Tom Date: Tue, 14 Jan 2014 01:03:37 +0200 Subject: [PATCH 2/3] Fixed labeled menu icon button icon positioning. A .button.labeled within .ui.menu > .item has icon vertically aligned closer to bottom, not middle. This fixes it. --- src/collections/menu.less | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/collections/menu.less b/src/collections/menu.less index 59f8e8bba..c2ad22854 100755 --- a/src/collections/menu.less +++ b/src/collections/menu.less @@ -1787,6 +1787,9 @@ .ui.menu .item { font-size: 1rem; } +.ui.menu:not(.vertical) .item > .button.labeled > .icon { + padding-top: 0.6em; +} .ui.vertical.menu { width: 15rem; } From 81ac1c91bbf602b52b35cde2501b60895980d871 Mon Sep 17 00:00:00 2001 From: Tom Date: Tue, 14 Jan 2014 01:58:03 +0200 Subject: [PATCH 3/3] In-Menu input action button corrections. Action button within menu had some sizing and positioning issues. This attempts to fix it. Tested with large, normal, small menus; .button, .icon.button, .labeled.button. All 9 combos. --- src/collections/menu.less | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/collections/menu.less b/src/collections/menu.less index c2ad22854..b90379114 100755 --- a/src/collections/menu.less +++ b/src/collections/menu.less @@ -1790,6 +1790,10 @@ .ui.menu:not(.vertical) .item > .button.labeled > .icon { padding-top: 0.6em; } +.ui.menu:not(.vertical) .item .action.input > .button { + font-size: 0.8em; + padding: 0.55em 0.8em; +} .ui.vertical.menu { width: 15rem; } @@ -1806,6 +1810,13 @@ padding-bottom: 0.6em; padding-top: 0.6em; } +.ui.large.menu:not(.vertical) .item .action.input > .button { + font-size: 0.8em; + padding: 0.9em; +} +.ui.large.menu:not(.vertical) .item .action.input > .button > .icon { + padding-top: 0.8em; +} .ui.large.menu .dropdown.item .item { font-size: 1rem; }