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

Filip Fila noreply at phabricator.kde.org
Thu Aug 1 16:29:57 BST 2019


filipf added inline comments.

INLINE COMMENTS

> zzag wrote in main.qml:118
> There are some technical terms that should stay "technical," "color temperature" is one of those.

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?

REPOSITORY
  R119 Plasma Desktop

BRANCH
  nightcolor-kcm-tweaks (branched from master)

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/20190801/14b15e90/attachment.html>


More information about the Plasma-devel mailing list