fixes for PDF context menu (logseq#7690)

* fix(pdf): do not hide ctx menu when its blank area is clicked * fix: ensure ctx menu is shown when text is selected
ofer1992 · Dec 15, 2022 · 6237399 · 6237399
1 parent 07a7368
commit 6237399
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/main/frontend/extensions/pdf/highlights.cljs b/src/main/frontend/extensions/pdf/highlights.cljs
@@ -174,6 +174,7 @@
      {:ref      *el
       :style    {:top top :left left :visibility (if (and @*highlight-mode? new?) "hidden" "visible")}
       :on-click (fn [^js/MouseEvent e]
+                  (.stopPropagation e)
                   (when-let [action (.. e -target -dataset -action)]
                     (action-fn! action true)))}
 
@@ -599,9 +600,10 @@
                                                :properties {}})))]
 
            ;; show ctx menu
-           (set-tip-state! {:highlight hl-fn
-                            :selection selection
-                            :point     point}))))
+           (js/setTimeout (fn []
+                            (set-tip-state! {:highlight hl-fn
+                                             :selection selection
+                                             :point     point})))) 0))
 
      [(:range sel-state)])