https://bugs.kde.org/show_bug.cgi?id=434819

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDSINFO                   |REPORTED
         Resolution|WAITINGFORINFO              |---

--- Comment #8 from Nate Graham <n...@kde.org> ---
I see from your screenshots how it was better with Oxygen, but the text is
still readable with Breeze, even if it's not quite *as* readable.

Making the shadows unconditional is a completely trivial change; remove or
comment out "root.isContainment && main.GridView.view.isRootView &&" in line
550 of FolderItemDelegate.qml.

However the result looks terrible with Breeze and I assume any other
mostly-opaque theme. So that's not an option.

That leaves a few other options I can think of, in no particular order:
1. Make it a user-facing option (ew)
2. Make it a theme-facing option such that highly transparent themes could
toggle it
3. Do some fancy theme introspection to toggle it automatically based on the
opacity level of the theme's background
4. Do nothing and say, "sorry this is too niche a use case to care about
upstream, and you can patch it out yourself or fork the applet on store.kde.org
if you disagree"

I don't feel entirely comfortable making this decision myself so I would like
to punt it to Eike. :)

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to