Review Request 124872: Default to Breeze colors

[Martin Gräßlin]

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

Ship it!


I must say I'm rather shocked at how this is implemented. Would it be possible to move the defaults into our QPT plugin, so that these defaults are only applied if running on a platform with our plugin, otherwise it picks Qt defaults?

- Martin Gräßlin


On Aug. 21, 2015, 11:35 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124872/
> -----------------------------------------------------------
> 
> (Updated Aug. 21, 2015, 11:35 p.m.)
> 
> 
> Review request for KDE Frameworks, Plasma and KDE Usability.
> 
> 
> Bugs: 351530
>     https://bugs.kde.org/show_bug.cgi?id=351530
> 
> 
> Repository: kconfigwidgets
> 
> 
> Description
> -------
> 
> This makes KColorScheme return Breeze colors by default, ie. choosing "Default" in Colors KCM is no longer Oxygen.
> 
> 
> Diffs
> -----
> 
>   src/kcolorscheme.cpp f1ec1e3 
> 
> Diff: https://git.reviewboard.kde.org/r/124872/diff/
> 
> 
> Testing
> -------
> 
> Seems to look right, typos reserved :)
> 
> Only thing missing now is the window decoration which is set in the WM section of kdeglobals by the breeze.colors but KWin defaults to highlight colors, so it stays blue.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150824/d5180e16/attachment.html>