fix: Escape does not close CMD+K when viewing a document or collection

2024-06-14 22:56:39 -04:00
parent 9415a35795
commit 6318714aee
2 changed files with 6 additions and 0 deletions
--- a/app/components/Sharing/Collection/SharePopover.tsx
+++ b/app/components/Sharing/Collection/SharePopover.tsx
@@ -59,6 +59,9 @@ function SharePopover({ collection, visible, onRequestClose }: Props) {
  useKeyDown(
    "Escape",
    (ev) => {
+      if (!visible) {
+        return;
+      }
      ev.preventDefault();
      ev.stopImmediatePropagation();

--- a/app/components/Sharing/Document/SharePopover.tsx
+++ b/app/components/Sharing/Document/SharePopover.tsx
@@ -67,6 +67,9 @@ function SharePopover({
  useKeyDown(
    "Escape",
    (ev) => {
+      if (!visible) {
+        return;
+      }
      ev.preventDefault();
      ev.stopImmediatePropagation();