cfeck added a comment.
In D11262#237017 <https://phabricator.kde.org/D11262#237017>, @Pitel wrote: > I guess I should have renamed it from `isLightColor` to something like `isNotAlmostBlack`... The point was to replace almost black colors with real black because it looks better with opacity. IMHO `(0,0,128)`, which is deep blue, is far enough from black to not mess with it because font colors readable on deep blue background might not be readable on black one. Ah okey. I had the impression this was used to decide if light or dark text needs to be used. I wouldn't want to see black text on dark blue buttons :) REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D11262 To: Pitel, mart, #plasma Cc: ragreen, fvogt, cfeck, fredrik, ngraham, abetts, broulik, mvourlakos, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
