D22863: [kcms/nightcolor] Minor UI tweaks to the KCM

Vlad Zagorodniy noreply at phabricator.kde.org
Fri Aug 2 14:43:30 BST 2019


zzag added inline comments.

INLINE COMMENTS

> filipf wrote in main.qml:118
> I think using the technical term here is fine. IIRC f.lux tried to simplify this by adding a bunch of descriptors which compared a particular temperature value or even range to a lighting condition, e.g. one generated by incandescent bulbs. But I don't think we need this, users will figure it out.
> 
> Regarding using uppercase c in "Night Color", that's 2 in favor and 2 against it. We had this dilemma with "Look and Feel" before and kept "Feel" capitalized because it's the name of a specific feature I guess. I'm not super passionate about what we choose, but I slightly lean towards leaving "Color" capitalized for now. That okay?

This option talks about color temperature, so lower case makes sense.

This is not a big deal, but just a little detail.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D22863

To: filipf, #plasma, #vdg, ngraham
Cc: zzag, romangg, GB_2, plasma-devel, LeGast00n, jraleigh, fbampaloukas, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190802/d3850790/attachment.html>


More information about the Plasma-devel mailing list