`7056e22`

Fix session exit cleanup: unmap windows and kill picom

- Remove glx-no-rebind-pixmap from picom config (prevents wallpaper change detection)
- Unmap all managed windows before exit so they don't persist on X server
- Kill picom on exit to prevent compositor effects bleeding into greeter
- Add sync to ensure X server processes unmap requests before exit

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 3 months ago

SHA: 7056e221e89346e232c830cc767b040c99586d3b
Parents: 78c6ca7
Tree: 4b3aa69

2 changed files

Status	File	+	-
M	`gar/src/config/mod.rs`	0	1
M	`gar/src/x11/events.rs`	21	1

gar/src/config/mod.rsmodified

  vsync = true;
  use-ewmh-active-win = true;
  glx-no-stencil = true;
 -glx-no-rebind-pixmap = true;
  # Rounded Corners
  corner-radius = {};

gar/src/x11/events.rsmodified

                  self.reload_config()?;
+             }
              Action::Exit => {
 -                tracing::info!("Exit requested");
 +                tracing::info!("Exit requested, will exit event loop");
                  self.running = false;
 +                // Force an immediate return from event handling
 +                return Ok(());
+             }
              Action::ToggleFloating => {
                  if let Some(window) = self.focused_window {
              std::thread::sleep(std::time::Duration::from_millis(10));
+         }
 +        // Unmap all managed windows so they don't persist on the X server
 +        // This ensures windows aren't visible when returning to the greeter
 +        tracing::info!("Unmapping {} managed windows", self.windows.len());
 +        for &window in self.windows.keys() {
 +            tracing::debug!("Unmapping window {}", window);
 +            let _ = self.conn.conn.unmap_window(window);
 +        }
 +        // Sync to ensure X server processes all unmap requests before we exit
 +        let _ = self.conn.sync();
 +        tracing::info!("Windows unmapped and synced");
++
          // Stop garbar if it was spawned
          if let Some(ref mut child) = self.garbar_process {
              stop_garbar(child);
+         }
          self.garbar_process = None;
 +        // Kill picom to prevent compositor effects from bleeding into the greeter
 +        tracing::info!("Killing picom...");
 +        let _ = std::process::Command::new("pkill")
 +            .arg("-x")
 +            .arg("picom")
 +            .status();
++
          // Signal systemd that graphical session has ended
          // This stops user services bound to graphical-session.target (like garbg)
          stop_graphical_session();