| Summary: | logout confirmation dialog is unreadable with certain light backgrounds and some 3rd party themes | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | luca <luca.pedrielli> |
| Component: | Theme - Breeze | Assignee: | visual-design |
| Status: | RESOLVED DOWNSTREAM | ||
| Severity: | minor | CC: | husek, nate, plasma-bugs |
| Priority: | NOR | Keywords: | usability |
| Version: | 5.12.5 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Debian testing | ||
| OS: | Linux | ||
| See Also: | https://bugs.kde.org/show_bug.cgi?id=382264 | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
screenshot
screenshot2 screenshot3 - breeze desktop theme |
||
Can you provide a screenshot? Created attachment 112651 [details]
screenshot
here it is
Thanks. Looks like the lightness calculation that was changed for Bug 382264 is mis-perceiving this background as dark instead of light. Perhaps because of the black terminal window? We should just unconditionally darken everything IMHO instead of playing color games like this. The color which decides whether make the background darker or lighter is the button background -- the color of OK button on the screenshot -- which is some kind of light blue so it is considered light background since the mentioned patch. IMHO even the buttons are hard to read which seems to be the root of the problem to me. Or is this color combination intentional? Right, the issue in the screenshot is that the buttons are hard to read because their text is white, and also the leave options buttons are practically invisible because they're white against a light background. Even if we fixed the button text to be black in this case, IMHO because the leave options are always white, we need to always darken, never lighten. Whenever we lighten, the leave options buttons are going to be unreadable. Does that make sense? Created attachment 112653 [details]
screenshot2
desktop with simple white color background
button background seems to be transparent That's the thing, leave options are not always white but their color is foreground color of complementary color group... Yes the background seems to be transparent. I didn't know it was possible / supported. Which desktop theme are you using? I've noticed this issue with "glassified" and "ktrans" desktop theme. Seems ok with breeze desktop themes. Can you attach a screenshot with regular Breeze? Created attachment 112660 [details]
screenshot3 - breeze desktop theme
Thanks. I'm tempted to mark this as RESOLVED DOWNSTREAM since it's caused by 3rd-party themes using transparency in a somewhat inappropriate manner... Yeah, this is a theme issue. It's not appropriate to use such heavy transparency for button backgrounds, since buttons need to be readable no matter what they're placed over. Please report this issue to the maintainers of the affected themes. Thanks! |
after 5.12.5 plasma update the logout confirmation dialog is white with white buttons and white font color. as workaround I changed /usr/share/plasma/look-and-feel/org.kde.breeze.desktop/contents/logout/Logout.qml //color: root.isLightColor(PlasmaCore.ColorScope.backgroundColor) ? PlasmaCore.ColorScope.backgroundColor : "black" color: "black"