From fcf9d571f0a9f1dd88fa9be44a7ba26634fba432 Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Wed, 3 May 2023 15:01:51 +0200 Subject: [PATCH] fix(AppMenu): On bright color theme the app icons in the overflow menu must be inverted Signed-off-by: Ferdinand Thiessen Signed-off-by: nextcloud-command --- core/src/components/AppMenu.vue | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/core/src/components/AppMenu.vue b/core/src/components/AppMenu.vue index ed3a6293c57..c4ea41ecc84 100644 --- a/core/src/components/AppMenu.vue +++ b/core/src/components/AppMenu.vue @@ -247,14 +247,18 @@ $header-icon-size: 20px; } ::v-deep .app-menu-more .button-vue--vue-tertiary { - color: var(--color-primary-text); opacity: .7; margin: 3px; filter: var(--background-image-invert-if-bright); - &:hover { - opacity: 1; - background-color: transparent !important; + /* Remove all background and align text color if not expanded */ + &:not([aria-expanded="true"]) { + color: var(--color-primary-text); + + &:hover { + opacity: 1; + background-color: transparent !important; + } } &:focus-visible { @@ -267,6 +271,8 @@ $header-icon-size: 20px; .app-icon { position: relative; height: 44px; + /* Icons are bright so invert them if bright color theme == bright background is used */ + filter: var(--background-invert-if-bright); &.has-unread::after { background-color: var(--color-main-text);