Deal with not receiving enternotify events

1 files changed, 10 insertions, 30 deletions

diff --git a/src/clients/mod.rs b/src/clients/mod.rs
index c42a385..bd44f1d 100644
--- a/src/clients/mod.rs
+++ b/src/clients/mod.rs
@@ -1,10 +1,9 @@
 //! Tracking and managing windows.
 
-use std::cmp::{min, Ordering};
+use std::cmp::Ordering;
 
 use crate::{
     buttons,
-    config::BORDER_WIDTH,
     conn_info::Connection,
     debug,
     error::{Error, Result},
@@ -156,47 +155,27 @@ impl ClientState {
             }
         }
 
-        let Ok(geom) = conn.wait_for_reply(conn.send_request(&GetGeometry {
-            drawable: Drawable::Window(window),
-        })) else {
-            return; // window stopped existing, so we can't manage it
-        };
-
+        let new_idx = self.clients.len();
         self.clients.push(Client::new(window, tag));
 
-        let mon_geom @ MonitorGeometry {
-            width: mon_width,
-            height: mon_height,
-            ..
-        } = self.focused_mon().screen_info;
+        let mon_geom = self.focused_mon().screen_info;
 
-        let c = &mut self.clients[0];
-        #[allow(clippy::cast_sign_loss)]
-        c.set_geom(
-            conn,
-            geom.x(),
-            geom.y(),
-            min(geom.width(), mon_width.saturating_sub(geom.x() as u16)),
-            min(geom.height(), mon_height.saturating_sub(geom.y() as u16)),
-            BORDER_WIDTH,
-        );
+        let c = &mut self.clients[new_idx];
         c.set_border(conn, conn.colours.border_normal());
-
         if floating {
             c.set_floating(conn);
         }
 
         c.sync_hints(conn, true);
         c.apply_geometry_hints(conn, &mon_geom);
-
-        c.ensure_mapped(conn);
-
         c.set_event_mask(
             conn,
             EventMask::ENTER_WINDOW
                 | EventMask::FOCUS_CHANGE
                 | EventMask::PROPERTY_CHANGE
-                | EventMask::STRUCTURE_NOTIFY,
+                | EventMask::STRUCTURE_NOTIFY
+                | EventMask::BUTTON_PRESS
+                | EventMask::KEY_PRESS,
         );
 
         // Add to net_client_list
@@ -207,10 +186,11 @@ impl ClientState {
             r#type: x::ATOM_WINDOW,
             data: &[window],
         });
-        c.set_withdrawn(conn, false);
 
-        self.refocus(conn, self.clients.len() - 1);
         self.rearrange(conn);
+        if self.focused_mon().focused_tag.matches(tag) {
+            self.refocus(conn, self.clients.len() - 1);
+        }
     }
 
     /// Stop managing the given destroyed window