D8159: Add API for setting server decoration palettes

[David Edmundson]

davidedmundson added a comment.


  If it does go into wayland-protocols I imagine it would get renamed?
  
  And even if it did get upstreamed and adopted, Steam (for example) would want to set the palette.

INLINE COMMENTS

> graesslin wrote in server_decoration_interface.h:147
> Nothing better than looking into the source:
> 
>   m_colorScheme = QStandardPaths::locate(QStandardPaths::GenericConfigLocation, colorScheme);

Nothing better, and it's what I did, but see the more relevant line above.

  : m_colorScheme(QFileInfo(colorScheme).isAbsolute()
                  ? colorScheme
                  : QStandardPaths::locate(QStandardPaths::GenericConfigLocation, colorScheme))

Then only go into the branch you looked at if the scheme is "kdeglobals".

It's both names and paths. Which is what the XML docs and unit test cover.

REPOSITORY
  R127 KWayland

REVISION DETAIL
  https://phabricator.kde.org/D8159

To: davidedmundson, #plasma
Cc: graesslin, broulik, plasma-devel, #frameworks, leezu, ZrenBot, alexeymin, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171006/62b4be60/attachment.html>