There's one massive problem we're going to face.

kwayland-integration is released with plasma
we have people using new frameworks with old plasma

porting breeze is fine, as that's in sync with kwayland integration, but I don't know of any nice way to do plasma dialogs in a way that won't introduce a regression

I don't know of any nice way to do plasma dialogs in a way that won't introduce a regression

I don't think there is a nice way to work around it. I'm afraid those people will have to stick with shadow-less plasma dialogs on wayland for quite some time. :(

While I was porting Oxygen to KWindowShadow yesterday, I encountered a problem or rather noticed it in the existing port of Breeze to KWindowShadow.

Currently, neither Oxygen nor Breeze try to destroy shadows when a widget becomes hidden or the platform window is destroyed. This will be not okay with KWindowShadow though. If you do new, then there has to be a delete somewhere... Not quite sure what to do about it. The current API is very sensible imho, but this intentional "resource leak" that Oxygen and Breeze do is just weird...

In T12496#216834, @zzag wrote:

I don't know of any nice way to do plasma dialogs in a way that won't introduce a regression

I don't think there is a nice way to work around it. I'm afraid those people will have to stick with shadow-less plasma dialogs on wayland for quite some time. :(

Hmm, so the best way to handle this is to land the plasma-framework patch when KF 5.70 is around the corner.

KDE Plasma 5.19 depends on KF 5.70.