Please, excuse me for shameless plug, but I am currently working on a "new" tiling script extension to KWin here. I've observed, that the other tiling extensions' development was stale over the year and decided to fork one of them with the goal of making the code base more modern and new developer friendly. Also, I am aiming for the Wayland support and wanted to publish the script on the KDE Store once that's ready, but my Wayland session is not stable right now on Plasma 5.22, and I haven't found a good way to debug the script right now in the VM (I just want to see script logs for now). In the end, I really want the script to be an official KDE project and to be a part of default Plasma distribution once it's mature enough (don't want to be a perfectionist, so I am not sure where to draw a line here).

See D29749

OK, let's wait a bit more until GIMP transitions. Still I would like to combine the GTK2 and GTK3 themes selection into one combobox or a list.

Moved here: https://invent.kde.org/plasma/kwin/-/merge_requests/39

Moved here: https://invent.kde.org/plasma/kde-gtk-config/-/merge_requests/2

I have a better idea of how to get rid of the dbus calls entirely in the future.

In D29749#671032, @ngraham wrote:

1. There's still GIMP (and IIRC Audacity) when it comes to major GTK2 apps. I'm not saying don't do this, but maybe it's a bit premature?

In D28623#666890, @trmdi wrote:

Just a question, SierraBreezeEnhanced has a feature called "Unison hovering". Does this support it?

In D28623#666890, @trmdi wrote:

Just a question, SierraBreezeEnhanced has a feature called "Unison hovering". Does this support it?

• Add support for aurorae themes
• Plastic does not work, but in that case decoration buttons in GTK apps are Breeze

In D28623#666400, @cblack wrote:

While this is neat, I'm fairly sure that creating the CSS for this in a manner that works across themes is going to be hard if not impossible.

Export buttons with fixed geometry (Now all buttons are exported with the same size)

Support the maximize button in the checked state

• Render buttons to SVGs

• Make buttons work with Breeze theme by respecting pallete.

• Use unique_ptr for buttons
• Propagate events to buttons

Is it possible to integrate the search into the help menu, if it exists already or create an entry for it. Much like macOS does.

• Export decorations as png images

So, is it ok to land? If so, approve please.

No objections from me.

We also need to remove the remaining GTK stuff from here once D28070 and D28072 are landed. Does anyone else wants to test the later?

Hm, this is bizarre. I've tested once again and now the bug I mentioned before is gone.

In D28072#638800, @gikari wrote:

I also noticed a dynamic recoloration bug: it doesn't work if the app is launched with the color scheme, that it was using in the previous launch. To make it work I have to change color scheme and relaunch the app. The app therefore changes its color scheme to a new one and it is now reacting to color scheme change properly (i.e. changes it without restart).

In D28072#643693, @ngraham wrote:

Still doesn't work for me:

gnome-chess --gtk-module /home/nate/kde/usr/lib64/gtk-3.0/modules/libcolorreload-gtk-module.so
Gtk-Message: 08:49:51.962: Failed to load module "colorreload-gtk-module"

• Add missed Licence header
• Remove unnecessary CMake commands

• Rebase on master

@ngraham Did you manage to load the recoloration module into the GTK app by one or another way?

• Remove explicit QVariant wrappers

Ping

In D28072#638514, @ngraham wrote:

Same error about not being able to find the module. Sorry for not being clear.

In D28072#638356, @ngraham wrote:

The path would be /home/nate/kde/usr/lib64/gtk-3.0/ on my system, but nonetheless there is no effect. :(

In D28072#638170, @ngraham wrote:

Hmm, when following those instructions, I get the following console error:

Gtk-Message: 08:50:13.086: Failed to load module "colorreload-gtk-module"

And the color scheme doesn't change. The colorreload-gtk3-module target did get built, and it installed a shared library to /home/nate/kde/usr/lib64/gtk-3.0/modules/libcolorreload-gtk-module.so

In D28072#638173, @cblack wrote:

GTK™ isn't too fond of trying to load stuff from outside the standard directories.

• Rebase on master
• Add recoloration module check

In D28360#636687, @kossebau wrote:

Edit: and yes, I agree CMake has awful flaws here when it comes to our needs to properly communicate dependencies to fellow developers & packagers. I am trying to give you best practices done so far elsewhere in KDE software to deal with this, to ensure consistent patterns in the approach to the same problem :)

Everything is fine on CI.

In D28360#636610, @kossebau wrote:

Edit: The proposed solution to make it simply required only forces packagers to add that and all what it pulls in as build-time dependency to their package creation, even if not needed for the package creation itself at all, which is not something packagers like (wastes resources on package build servers).

In D28360#636435, @eszlari wrote:

In D28360#636418, @gikari wrote:

But the option is enabled by default and needs to be explicitly disabled. This seems to be an overengineered solution. It basically says, that XSettingsd is an optional dependency, which already is.

But it make the packager aware, that this dependency exists, instead of at runtime, the end user wondering, why his/her Plasma desktop behaves differently from others.

Anecdote: obs-studio checks for pulseaudio at compile time, but fails silently(!) when it can't find it, leading users to wonder, why they can't record audio.

But the option is enabled by default and needs to be explicitly disabled. This seems to be an overengineered solution. It basically says, that XSettingsd is an optional dependency, which already is.

In D28360#636404, @eszlari wrote:

I would like to have this in 5.18

Why just not make xsettingsd required?

Ok, let's keep our eyes on CI now. Everything should be fine, since the dependencies didn't change.

See https://invent.kde.org/sysadmin/ci-tooling/-/merge_requests/49 for possible resolution

This has been reverted. @cblack, please take a look at what has happened. Seems like CI is missing some dependency.

Rebase on master