From d6a2ac86cbc6125739af02485eb755a35c0db532 Mon Sep 17 00:00:00 2001 From: Tim McConechy Date: Thu, 9 May 2024 11:54:10 -0400 Subject: [PATCH 1/4] Fix app menu buttons --- src/components/applicationmenu/_applicationmenu-new.scss | 4 ++++ src/components/button/_button-new.scss | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/components/applicationmenu/_applicationmenu-new.scss b/src/components/applicationmenu/_applicationmenu-new.scss index ac8a039abc..5ab38de877 100644 --- a/src/components/applicationmenu/_applicationmenu-new.scss +++ b/src/components/applicationmenu/_applicationmenu-new.scss @@ -184,6 +184,10 @@ color: inherit; top: auto; + &:hover { + background-color: unset !important; + } + &:focus:not(.hide-focus) { border-color: $new-appmenu-selected-border-color; box-shadow: none; diff --git a/src/components/button/_button-new.scss b/src/components/button/_button-new.scss index 8d4d0f1a21..56c0311507 100644 --- a/src/components/button/_button-new.scss +++ b/src/components/button/_button-new.scss @@ -185,7 +185,7 @@ &.close, &.btn-filter, &.personalize-actionable { - background-color: unset; + background-color: unset !important; color: unset; } From 631f7f015342c42bb905d154e1e24b14df0e7d28 Mon Sep 17 00:00:00 2001 From: Tim McConechy Date: Fri, 10 May 2024 11:59:12 -0400 Subject: [PATCH 2/4] Fix for classic mode --- .../applicationmenu/_applicationmenu-new.scss | 2 +- src/components/applicationmenu/_applicationmenu.scss | 10 ++++++++++ src/themes/theme-classic-contrast.scss | 4 ++-- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/src/components/applicationmenu/_applicationmenu-new.scss b/src/components/applicationmenu/_applicationmenu-new.scss index 5ab38de877..71a56b1f56 100644 --- a/src/components/applicationmenu/_applicationmenu-new.scss +++ b/src/components/applicationmenu/_applicationmenu-new.scss @@ -195,7 +195,7 @@ } .icon { - top: auto; + top: -2px; &.plus-minus { &::before, diff --git a/src/components/applicationmenu/_applicationmenu.scss b/src/components/applicationmenu/_applicationmenu.scss index 0cd80e08cc..c5f54d3529 100644 --- a/src/components/applicationmenu/_applicationmenu.scss +++ b/src/components/applicationmenu/_applicationmenu.scss @@ -724,3 +724,13 @@ html[dir='rtl'] .application-menu .hyperlink { display: flex; justify-content: center; } + +.application-menu .accordion.panel.inverse .accordion-header .btn:hover, +.application-menu .accordion.panel.inverse .accordion-header .btn-expander:hover { + background-color: transparent !important; + color: #fff !important; +} + +.accordion.panel.inverse .accordion-header:hover .icon { + color: #fff !important; +} diff --git a/src/themes/theme-classic-contrast.scss b/src/themes/theme-classic-contrast.scss index 02d75da892..7ebb90eed2 100755 --- a/src/themes/theme-classic-contrast.scss +++ b/src/themes/theme-classic-contrast.scss @@ -499,8 +499,8 @@ $accordion-inverse-focus-state-color: $ids-color-palette-white; $accordion-focused-text-color: $ids-color-palette-azure-50; $accordion-focused-border-color: $ids-color-palette-azure-50; -$accordion-selected-bg-color: $ids-color-palette-slate-40; -$accordion-selected-text-color: $ids-color-palette-graphite-90; +$accordion-selected-bg-color: $ids-color-palette-slate-100; +$accordion-selected-text-color: $ids-color-palette-graphite-20; $accordion-highcontrast-text-decoration: underline; From fbcfd44eed21d0dc58636acedd3aa7afb4dbfe21 Mon Sep 17 00:00:00 2001 From: Tim McConechy Date: Mon, 13 May 2024 10:02:11 -0400 Subject: [PATCH 3/4] Fix hover state --- src/components/applicationmenu/_applicationmenu.scss | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/components/applicationmenu/_applicationmenu.scss b/src/components/applicationmenu/_applicationmenu.scss index c5f54d3529..6c6b213499 100644 --- a/src/components/applicationmenu/_applicationmenu.scss +++ b/src/components/applicationmenu/_applicationmenu.scss @@ -734,3 +734,7 @@ html[dir='rtl'] .application-menu .hyperlink { .accordion.panel.inverse .accordion-header:hover .icon { color: #fff !important; } + + .btn-icon.close:hover:not([disabled]) { + background-color: transparent !important; + } From dca2dd50da73818579879040c3ac033fc213ac1b Mon Sep 17 00:00:00 2001 From: Tim McConechy Date: Tue, 14 May 2024 10:11:41 -0400 Subject: [PATCH 4/4] Add additional fixes --- src/components/applicationmenu/_applicationmenu.scss | 12 ++++++++++-- src/components/module-nav/module-nav.accordion.scss | 4 ++++ 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/src/components/applicationmenu/_applicationmenu.scss b/src/components/applicationmenu/_applicationmenu.scss index 6c6b213499..82a154af06 100644 --- a/src/components/applicationmenu/_applicationmenu.scss +++ b/src/components/applicationmenu/_applicationmenu.scss @@ -735,6 +735,14 @@ html[dir='rtl'] .application-menu .hyperlink { color: #fff !important; } - .btn-icon.close:hover:not([disabled]) { +.application-menu .btn-icon.close:hover:not([disabled]) { background-color: transparent !important; - } +} + +.application-menu .btn:hover:not([disabled]) { + background-color: rgba(255, 255, 255, .1) !important; +} + +.application-menu .btn-icon:hover:not([disabled]) { + background-color: rgba(255, 255, 255, .1) !important; +} diff --git a/src/components/module-nav/module-nav.accordion.scss b/src/components/module-nav/module-nav.accordion.scss index a2a3f3c8bb..125220c0cd 100644 --- a/src/components/module-nav/module-nav.accordion.scss +++ b/src/components/module-nav/module-nav.accordion.scss @@ -472,3 +472,7 @@ } } } + +.module-nav-main.accordion-section .btn:hover:not([disabled]) { + background-color: transparent !important; +}