Handle unmanaged windows appropriately

2 files changed, 11 insertions, 1 deletions

diff --git a/sway/handlers.c b/sway/handlers.c
index 59e67f59..f1098835 100644
--- a/sway/handlers.c
+++ b/sway/handlers.c
@@ -33,6 +33,7 @@ bool handle_view_created(wlc_handle view) {
 }
 
 void handle_view_destroyed(wlc_handle view) {
+        sway_log(L_DEBUG, "Destroying window %d", view);
         destroy_view(get_swayc_for_handle(view, &root_container));
         return true;
 }
@@ -53,7 +54,6 @@ bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
         
         bool ret = true;
         struct sway_mode *mode = config->current_mode;
-        sway_log(L_DEBUG, "key pressed: %d %d", sym, modifiers->mods);
 
         // Lowercase if necessary
         sym = tolower(sym);
diff --git a/sway/layout.c b/sway/layout.c
index 403ec13a..2a57ccce 100644
--- a/sway/layout.c
+++ b/sway/layout.c
@@ -121,6 +121,12 @@ swayc_t *get_focused_container(swayc_t *parent) {
 }
 
 void add_view(wlc_handle view_handle) {
+        const uint32_t type = wlc_view_get_type(view_handle);
+        if (type & WLC_BIT_UNMANAGED) {
+                sway_log(L_DEBUG, "Leaving view %d alone (unmanaged)", view_handle);
+                return;
+        }
+
         swayc_t *parent = get_focused_container(&root_container);
         sway_log(L_DEBUG, "Adding new view %d under container %p %d", view_handle, parent, parent->type);
 
@@ -142,6 +148,10 @@ void add_view(wlc_handle view_handle) {
 }
 
 void destroy_view(swayc_t *view) {
+        if (view == NULL) {
+                sway_log(L_DEBUG, "Warning: NULL passed into destroy_view");
+                return;
+        }
         sway_log(L_DEBUG, "Destroying container %p", view);
         swayc_t *parent = view->parent;
         if (!parent) {