<table><tr><td style="">zzag added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21948">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>How are they orthogonal?</p></blockquote>

<p>You're asking to redesign color correction infrastructure while this patch adds sort of unrelated feature.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I tell you that's not a part of Night Color, because Night Color as the name says deals with a specific time of day: night,</p></blockquote>

<p>This argument is not convincing. The Night Color manager is a blue light filter. If the name is main blocker, let's rename the manager. "Blue Light Filter" is a good candidate, it's generic and accurate.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>And reading your sentence it is still not clear to me why we "need" a switch.</p></blockquote>

<p>Because the Night Color manager has to be responsible for management of screen color temperature and all three existing modes are dynamic.</p>

<p>Introducing separate path for the constant mode is a bad idea because two actors will be involved in management of color temperature, which means more complexity. Managing those two actors will make life harder for us developers and for clients. On server side, we would have to take some extra measures to ensure that there are no conflicts between the Night Color manager and the constant mode implementation. On client side, it means two different apis to interact with.</p>

<p>Another problem with two paths is that it just doesn't make sense conceptually...</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>You're right, lets step it back to discussing purely in terms of UX, get on the same page then that will direct the code.</p></blockquote>

<p>Okay, that's a good idea.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21948">https://phabricator.kde.org/D21948</a></div></div><br /><div><strong>To: </strong>zzag, KWin<br /><strong>Cc: </strong>davidedmundson, romangg, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart<br /></div>