----------------------------------------------------------- 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
