D11262: KDE logout screen background color fix
Marco Martin
noreply at phabricator.kde.org
Fri Mar 23 13:13:12 UTC 2018
mart added a comment.
In D11262#224843 <https://phabricator.kde.org/D11262#224843>, @Pitel wrote:
> I see that relative luminance formula is more correct but I would like to point out that we do not need here to calculate precise luminance. The point of the `?:` statement is to collapse colors close to back to real black because gray overlay with 50 % opacity does not look good. So the current formula looks good enough for me and I would even think about moving the threshold down a bit to 0.4 or 0.3. (Generally not collapsing to black is the save way, it may not look so good but the text will be still easy to read.)
yep, gray background definitely doesn't look good.
can you try with the luminance formula and put a screenshot on how it looks?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D11262
To: Pitel, mart, #plasma
Cc: fredrik, ngraham, abetts, broulik, mvourlakos, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180323/e1a225e0/attachment.html>
More information about the Plasma-devel
mailing list