Skip to content

Ensure we can set per-window properties before the intial commit

David Edmundson requested to merge work/initial into master

In the current implementation we cannot use a LayerShellQt before the shell surface is created.

At the moment a shell surface is created, the constructor is run and then QtWayland commits the current state. This means the compositor configures the window before a client has any chance to set anchors or margins.

This works whilst we're just being a simple fullscreen window, but won't scale for plasmashell in the future.

This patch makes LayerShellQt::Window always creatable, and we can set and cache properties before the platform window is created, just like one can on QWindow and XDGShell properties.

This also makes it less potentially crashy as ::get always returns a valid result, and sets up the public API to be QML-able as an attached property in future.

Co-authored with Aleix's patch.

Merge request reports