Created attachment 114592 [details]
okular

Nate how should we solve this?

I was hoping you had some idea. :)

(In reply to Nate Graham from comment #3)
> I was hoping you had some idea. :)

no no
I was asking from a design standpoint
do we make the text lighter? or give it a background? what else?

Hah, I see! Seems like there is a general conceptual issue here:

When using Breeze light and the transparent+blurred element is over a dark element, the menu background darkens significantly, which reduces the readability of dark text (i.e. active menu items).

When using Breeze dark and the transparent+blurred element is over a light element, the background lightens significantly, which reduces the readability of light text (e.g. inactive menu items).


Here's an idea: When the transparent element's inherent color is significantly different from the color of whatever's beneath it, we could brighten or darken the result to make it better match the transparent element's original color.

I don't know what the code is like right now but we could tint disabled entries instead of making them transparent.

Created attachment 130469 [details]
opaque vs semi-transparent text

The best approach here is to add transparency to the disabled text as well (it is currently not transparent, just greyed out).
In the screenshot, the accelerator text has 60% opacity, while the left text is 100% opaque and nearly invisible.

A possibly relevant merge request was started @ https://invent.kde.org/plasma/breeze/-/merge_requests/22

This was fixed by that. :