https://bugs.kde.org/show_bug.cgi?id=449985
Nate Graham <n...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Version Fixed In| |5.24.2
Resolution|--- |FIXED
Latest Commit| |https://invent.kde.org/plas
| |ma/plasma-workspace/commit/
| |146ef22fcbdaf8bc4648e1fb29a
| |6415a524f83dd
Status|ASSIGNED |RESOLVED
--- Comment #12 from Nate Graham <n...@kde.org> ---
Git commit 146ef22fcbdaf8bc4648e1fb29a6415a524f83dd by Nate Graham.
Committed on 17/02/2022 at 18:25.
Pushed by ngraham into branch 'master'.
Lock/login themes: stop adjusting shadows based on color scheme
Right now we change the intensity of shadows based on the color scheme's
text color, but with a property called "lightBackground". This indicates
that we wanted to do it based on the background color itself, but did
not or could not, so settled on using the text color as a proxy. However
this is wrong; there is no relationship between the active color
scheme's text color and the color of the background image.
Even if we could or did fix that, it would probably still be wrong,
because using lighter shadows on a genuinely light background would
result in worse contrast and readability, especially if the light
background was very visually busy.
For these reasons, in most other places, we have since settled on the
mantra of "shadows should be black", and this commit makes the lock and
login screens follow that convention as well.
FIXED-IN: 5.24.2
M +1 -7 lookandfeel/contents/lockscreen/LockScreenUi.qml
M +2 -13 sddm-theme/Main.qml
https://invent.kde.org/plasma/plasma-workspace/commit/146ef22fcbdaf8bc4648e1fb29a6415a524f83dd
--
You are receiving this mail because:
You are watching all bug changes.
