`6cba17f`

Don't re-order hidden overlays on geometry sync

orderWindow:relativeTo: re-shows an off-screen window as a side effect.
sync_overlay called order_above whenever a target's bounds changed, which
defeated active_only mode for stacked windows: tarmac's stack peek-out
positions shift on every stack cycle, so each non-focused stack overlay
popped back onto the screen as a grey border.

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 1 week ago

SHA: 6cba17f48f98f2baf73494338a81a7c2b830f5c1
Parents: 1319fb0
Tree: dd13636

1 changed file

Status	File	+	-
M	`src/main.rs`	11	1

src/main.rsmodified

+             }
+         }
 +        let active_only = self.active_only;
 +        let focused = self.focused_wid;
++
          if let Some(overlay) = self.overlays.get_mut(&target_wid) {
              let prev = overlay.bounds();
              if size_changed(prev, bounds) || origin_changed(prev, bounds) {
                      bounds.size.width,
                      bounds.size.height,
                  );
 -                overlay.window.order_above(target_wid);
 +                // orderWindow:relativeTo: re-shows an off-screen window
 +                // as a side effect. In active_only mode, non-focused
 +                // overlays must remain hidden — otherwise stack peek
 +                // positions cause every stacked window's overlay to
 +                // pop onto the screen as their bounds shift.
 +                if !active_only || target_wid == focused {
 +                    overlay.window.order_above(target_wid);
 +                }
+             }
+         }