fix(wayland): layershell windows are recreated if attach fails

Usually this happens if the namespace changes

src/core/proxywindow.cpp

@@ -28,14 +28,7 @@ ProxyWindowBase::~ProxyWindowBase() {
 }
 
 void ProxyWindowBase::onReload(QObject* oldInstance) {
-	auto* old = qobject_cast<ProxyWindowBase*>(oldInstance);
+	this->window = this->createWindow(oldInstance);
-
-	if (old == nullptr || old->window == nullptr) {
-		this->window = new QQuickWindow();
-	} else {
-		this->window = old->disownWindow();
-	}
-
 	this->setupWindow();
 
 	Reloadable::reloadRecursive(this->mContentItem, oldInstance);
@@ -52,6 +45,16 @@ void ProxyWindowBase::onReload(QObject* oldInstance) {
 	this->window->setVisible(this->mVisible);
 }
 
+QQuickWindow* ProxyWindowBase::createWindow(QObject* oldInstance) {
+	auto* old = qobject_cast<ProxyWindowBase*>(oldInstance);
+
+	if (old == nullptr || old->window == nullptr) {
+		return new QQuickWindow();
+	} else {
+		return old->disownWindow();
+	}
+}
+
 void ProxyWindowBase::setupWindow() {
 	// clang-format off
 	QObject::connect(this->window, &QWindow::visibilityChanged, this, &ProxyWindowBase::visibleChanged);

src/core/proxywindow.hpp

@@ -55,6 +55,7 @@ public:
 
 	void onReload(QObject* oldInstance) override;
 
+	virtual QQuickWindow* createWindow(QObject* oldInstance);
 	virtual void setupWindow();
 
 	// Disown the backing window and delete all its children.

src/wayland/CMakeLists.txt

@@ -13,7 +13,6 @@ pkg_check_modules(wayland REQUIRED IMPORTED_TARGET wayland-client wayland-protoc
 find_package(Qt6 REQUIRED COMPONENTS WaylandClient)
 
 target_link_libraries(quickshell-wayland PRIVATE ${QT_DEPS} wayland-client)
-message(STATUS Qt6::WaylandClient)
 
 # wayland protocols
 

src/wayland/waylandlayershell.cpp

@@ -5,6 +5,7 @@
 #include <qobject.h>
 #include <qqmllist.h>
 #include <qquickitem.h>
+#include <qquickwindow.h>
 #include <qtmetamacros.h>
 #include <qtypes.h>
 
@@ -17,6 +18,31 @@ WaylandLayershell::WaylandLayershell(QObject* parent)
     : ProxyWindowBase(parent)
     , ext(new LayershellWindowExtension(this)) {}
 
+QQuickWindow* WaylandLayershell::createWindow(QObject* oldInstance) {
+	auto* old = qobject_cast<WaylandLayershell*>(oldInstance);
+	QQuickWindow* window = nullptr;
+
+	if (old == nullptr || old->window == nullptr) {
+		window = new QQuickWindow();
+	} else {
+		window = old->disownWindow();
+
+		if (this->ext->attach(window)) {
+			return window;
+		} else {
+			window->deleteLater();
+			window = new QQuickWindow();
+		}
+	}
+
+	if (!this->ext->attach(window)) {
+		qWarning() << "Could not attach Layershell extension to new QQUickWindow. Layer will not "
+		              "behave correctly.";
+	}
+
+	return window;
+}
+
 void WaylandLayershell::setupWindow() {
 	this->ProxyWindowBase::setupWindow();
 
@@ -32,10 +58,6 @@ void WaylandLayershell::setupWindow() {
 	QObject::connect(this, &WaylandLayershell::anchorsChanged, this, &WaylandLayershell::updateAutoExclusion);
 	QObject::connect(this, &WaylandLayershell::marginsChanged, this, &WaylandLayershell::updateAutoExclusion);
 	// clang-format on
-
-	if (!this->ext->attach(this->window)) {
-		// todo: discard window
-	}
 }
 
 void WaylandLayershell::setWidth(qint32 width) {

src/wayland/waylandlayershell.hpp

@@ -2,6 +2,8 @@
 
 #include <qobject.h>
 #include <qqmlintegration.h>
+#include <qquickitem.h>
+#include <qquickwindow.h>
 #include <qtmetamacros.h>
 #include <qtypes.h>
 
@@ -30,6 +32,7 @@ class WaylandLayershell: public ProxyWindowBase {
 public:
 	explicit WaylandLayershell(QObject* parent = nullptr);
 
+	QQuickWindow* createWindow(QObject* oldInstance) override;
 	void setupWindow() override;
 
 	void setWidth(qint32 width) override;