D11785: Add kcolorschemechooser support and related setting menu entry

Alexander Zhigalin noreply at phabricator.kde.org
Mon May 28 17:46:26 UTC 2018

zhigalin accepted this revision.
zhigalin added a subscriber: broulik.
zhigalin added a comment.
This revision is now accepted and ready to land.

  In D11785#245911 <https://phabricator.kde.org/D11785#245911>, @dhaumann wrote:
  > Indeed, maybe this class should be part of the KConfigWidgets framework? See https://api.kde.org/frameworks/kconfigwidgets/html/annotated.html for current list of classes - it already contains the color scheme stuff and is a tier3 framework (so many dependencies allowed).
  > @zhigalin Any comments from your side?
  Sorry for the late reply, haven't noticed this review before.
  As I said previously:
  > Now KDelelop also allows scheme selection, but I still think this should become a standard functionality, maybe something like `KStandardAction::configureColors(KXmlGuiWindow *win);`
  And this is a reply by @broulik
  > If, and only if, we were to add something like that, we could add it to KXmlGui and have it provide a settings dialog in the settings menu where you also find eg. "Configure Toolbars".
  > The infrastructure is already there, KWin can even follow the colorscheme of an application to colorize the title bar accordingly. However, with KColorSchemeManager there's an easy-to-use class that applications that want to provide such functionality (eg. Krita, KDevelop, Gwenview, Dragon Player) > can add to their settings without further complicating the already insane customizability of our applications.


> katecolorschemechooser.h:39
> +*/
> +class KateColorSchemeChooser : public QAction
> +{

`KateColorSchemeChooser` may be a bit misleading as there is also the Editor color scheme which is different...  
Maybe it should be like `KateGuiColorsChooser` or `KateWindowColorChooser`?
The rest looks good to me.

  R40 Kate


To: tfjellstrom, #kate, zhigalin
Cc: broulik, apol, kwrite-devel, zhigalin, dhaumann, ngraham, cullmann, mwolff, michaelh, kevinapavew, demsking, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180528/a69c95ca/attachment.html>

More information about the KWrite-Devel mailing list