Fix focus loss in customize popover after Reassign and other state transitions

When the Reassign button (or any focused element inside the feedback
banner) was removed from the DOM during a rec-state transition, browser
focus fell to document.body. The popover's scoped KeyHandler only
receives events that traverse its DOM subtree, so it went deaf to
subsequent keypresses — the user saw row flash animations instead of
conflict detection.

Two fixes:
1. Add a use-effect that re-focuses the popover element whenever
   rec-state changes and focus has drifted outside the popover.
2. After refresh! reinstalls global shortcut handlers, re-suppress
   them if a scoped key handler is still active (refcount > 0),
   preventing the 50ms window where global handlers could intercept
   keypresses meant for the popover.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2 people

src/main/frontend/components/shortcut.cljs

@@ -427,7 +427,13 @@
         (fn [new-binding]
           (let [binding' (if (= binding new-binding) nil new-binding)]
             (shortcut/persist-user-shortcut! k binding')
-            (js/setTimeout #(do (shortcut/refresh!) (saved-cb)) 50)))
+            (js/setTimeout #(do (shortcut/refresh!)
+                                ;; refresh! reinstalls global handlers unconditionally.
+                                ;; If a scoped key handler is active (popover open),
+                                ;; re-suppress to prevent the new handlers from firing.
+                                (when (pos? @*global-listener-refcount)
+                                  (shortcut/unlisten-all! true))
+                                (saved-cb)) 50)))
 
         cancel-fn!
         (fn []
@@ -638,6 +644,17 @@
                  (set-keystroke! #(util/trim-safe (str % kn)))))))))
      [*auto-accept-timer *fade-timer])
 
+    ;; Re-focus the popover when rec-state changes and focus has drifted outside.
+    ;; This handles the case where a focused element (e.g., the Reassign button)
+    ;; is removed from the DOM during a state transition, causing focus to fall
+    ;; to document.body and making the popover deaf to subsequent keypresses.
+    (hooks/use-effect!
+     (fn []
+       (when-let [el (rum/deref *ref-el)]
+         (when-not (.contains el (.-activeElement js/document))
+           (js/requestAnimationFrame #(.focus el)))))
+     [rec-state])
+
     ;; === V3 LAYOUT ===
     [:div.shortcut-popover
      {:tab-index -1