<table><tr><td style="">albertfreeman edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-aagtdyl32bzthy5/" rel="noreferrer">(Show Details)</a><br />albertfreeman added a subscriber: aacid.
</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/D8051" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">BUG: 182994<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Enable background color to be changed from settings in a way that by default preserves the Qt toolkit selection.<br />
<br />
Reasons for this change:<br />
    Youtube video with ~100 upvotes and ~60'000 views (although it changes the pages color instead of the background color)(https://www.youtube.com/watch?v=_N7PgpkH8lM)<br />
<br />
    So the code is useful for accessibility, eye strain and aesthetic reasons.<br />
<br />
    Additionally, power consumption is minorly affected by color and depends on display technology, sometimes black uses more power, othertimes white does<br />
     (https://www.quora.com/Does-a-white-background-use-more-energy-on-an-LCD-than-if-it-was-set-to-black)<br />
<br />
Reasons for this change by Henrik Fehlauer (rkflx at lab12.net):<br />
Thanks for providing a patch instead of just complaining on Bugzilla. As Reviewboard will become readonly in the near future, I would encourage you to bring this review over to Phabricator, so we can discuss further improvements (if you are still interested, which I hope, as the latest version of your patch is much improved).<br />
<br />
For example, you could improve the wording and layout:<br />
[ ] Use custom background color: [color chooser] (← Note inversion of checkbox logic, and disable colour chooser if checkbox is unchecked. Possibly decrease the colour chooser's width.)<br />
<br />
Advantages: Does away with the unmeaningful "Use UI theme", does not duplicate the words "background color" and is more compact (only one line) and less cluttered.<br />
<br />
@aacid:<br />
FWIW, I'm running a patched Okular with a different (hard coded) background colour since years, because basing the non-page background colour on the window background colour gives a very ugly result in my case (I don't think the code is to blame here, it's just impossible to find an algorithm which works for every possible colour). But I'm not alone, the following bugs have a total of 36 votes as of today:<br />
<br />
https://bugs.kde.org/show_bug.cgi?id=182994<br />
https://bugs.kde.org/show_bug.cgi?id=307116<br />
https://bugs.kde.org/show_bug.cgi?id=319736<br />
https://bugs.kde.org/show_bug.cgi?id=372055<br />
https://bugs.kde.org/show_bug.cgi?id=369627#c15<br />
<br />
Looking at other apps like Gwenview or Libreoffice, those often allow to change the non-page background colour, too.<br />
<br />
As for the maintenance argument, I don't think it holds: This is no new subsystem but a trivial feature with no complex code dependencies, and we are already showing a colour selection dialog and setting colours in other places in Okular.<br />
End of Reasons for this change by Henrik Fehlauer<br />
<br />
</span>https://git.reviewboard.kde.org/r/130219/<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8051" rel="noreferrer">https://phabricator.kde.org/D8051</a></div></div><br /><div><strong>To: </strong>albertfreeman, Okular<br /><strong>Cc: </strong>aacid, ltoscano, ngraham<br /></div>