<table><tr><td style="">romangg 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 #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D21948#487369" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D21948#487369</a>, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidedmundson</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I fully support a better split of colour correct and night mode, with nightmode being an interface into colour correction. <br />
 I didn't like really "nightmode" being an explicit part of kwin.</p></div>
</blockquote>

<p>I didn't say Night Color should be an interface only per se. Currently we do the timing calculation in KWin to always have the correct timings and color values at hand immediately without flashing frames with wrong temperature. To do this out of process we would need to block KWin's present on hardware until the temperature supplying process gives us the current value. Another aspect is when temperature changes over time. Do we have a timer in KWin or do we send progressively new values from the other process?</p>

<p>But it's worth investigating. Sun calculations don't really belong in a window compositor.</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>However imho we still want a constant toggle on the nightmode public api to integrate into something that can be feature complete with the current redshift plasmoid. Having that go into a different api would be messy and have potential conflicts at runtime.</p></blockquote>

<p>The redshift plasmoid targeted X and imo feature completeness with it shouldn't be a development target for us. The constant factor you can apply there is just for convenience on X because you never know what other ui you have in the DE the user runs for doing such things. In our Wayland session we can do this in an integrated way once instead of being spread over several little tools. And to be honest there is not much stopping us from doing it the same way on X11. KGamma just sucks.</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>