broulik added a comment.
lgtm
INLINE COMMENTS
> kwaylandintegration.cpp:72
> + if (paletteManagerInterface .name != 0) {
> + m_paletteManager =
> registry->createServerSideDecorationPaletteManager(paletteManagerInterface.name,
> paletteManagerInterface.version, this);
> + }
Can't we create these the first time an app sets a palette since hardly any
apps actually do?
REPOSITORY
R135 Integration for Qt applications in Plasma
REVISION DETAIL
https://phabricator.kde.org/D9658
To: davidedmundson, #plasma
Cc: broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,
jensreuterberg, abetts, sebas, apol, mart
