Skip to content

Dark mode icons are inverted in right click menu, but the correct ico…#15571

Open
Andreas-Kainz wants to merge 1 commit intomainfrom
private/Andreas-Kainz/DarkModeCSSIssue
Open

Dark mode icons are inverted in right click menu, but the correct ico…#15571
Andreas-Kainz wants to merge 1 commit intomainfrom
private/Andreas-Kainz/DarkModeCSSIssue

Conversation

@Andreas-Kainz
Copy link
Copy Markdown
Contributor

…ns are available

Change-Id: I3f1486c0402b776c2d5ee9b2654404a007f3cefb

image

I'm not sure cause this css rule is in cool since 3 years and I'm a bit surprize that it's not correct.

However, ordinary ALL collibre icons should be also available as dark variant. So an css invert isn't needed I think.

But you know css rules can change at different places, so please I don't know why this rule exist.

…ns are available

Signed-off-by: Andreas-Kainz <kainz.a@gmail.com>
Change-Id: I3f1486c0402b776c2d5ee9b2654404a007f3cefb
@eszkadev
Copy link
Copy Markdown
Contributor

It's not bout icons, it's about combobox entries which are "rendered" as images -> font list.
I guess we need more detailed selector to target only "renders" no "icons" inside comobbox

Comment thread browser/css/jsdialogs.css
border: 1px solid var(--color-primary);
}

[data-theme='dark'] .ui-combobox-entry img {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need that but for img:not( .... )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Review

Development

Successfully merging this pull request may close these issues.

Dark mode icons missing in context menu

2 participants