Review Request 130219: Enable background color to be changed from settings

[Albert Freeman]

> On Aug. 9, 2017, 6:24 p.m., Albert Astals Cid wrote:
> > Thanks for contributing to Okular :)
> > 
> > Why would you need a different background color?
> 
> Albert Freeman wrote:
>     I personally much prefer black as a background color.
> 
> Albert Astals Cid wrote:
>     Maybe you could edit your color scheme in system settings instead?

Through system settings I can change the okular background color but it also affects the window color of the rest of okular and all other applications. Additionally it seems to mix whatever color is selected with grey for the final background color, so if I set it as RGB 0, 0, 0 black it will appear a darker grey and if I set it RGB 255, 255, 255 white it will appear a lighter grey. The rest of the okular window does not mix with grey howevver. Should I try to make the system settings colors more granular in whichever kde codebase is responsible AND require an okular code change to adapt to that or just allow a choice of custom background color or system color scheme in okular in a better way than my attached code does (since my attached code is flawed)? Or should I just patch this manually to okular source code just on my machine?


- Albert


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/130219/#review103582
-----------------------------------------------------------


On Aug. 11, 2017, 2:50 a.m., Albert Freeman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/130219/
> -----------------------------------------------------------
> 
> (Updated Aug. 11, 2017, 2:50 a.m.)
> 
> 
> Review request for Okular.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> Enable background color to be changed from settings
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt ab7ad239 
>   conf/dlggeneral.h f363d260 
>   conf/dlggeneral.cpp 964a655b 
>   conf/dlggeneralbase.ui cf4ebca0 
>   conf/okular.kcfg 69ea8cf6 
>   mobile/components/CMakeLists.txt f1af2602 
>   ui/pageview.cpp 3d935a2e 
>   ui/pageviewutils.cpp a57712ca 
> 
> Diff: https://git.reviewboard.kde.org/r/130219/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Albert Freeman
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20170818/42522566/attachment.html>