Minor tweaks to how we decide when to re-generate context; speculativ…

…e fix for #185.
coddingtonbear · Apr 2, 2024 · abaef05 · abaef05
1 parent df8e46a
commit abaef05
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "obsidian-web",
-  "version": "3.3.3",
+  "version": "3.3.4",
   "description": "Connect your browser with your notes in Obsidian",
   "main": "index.js",
   "scripts": {

diff --git a/public/manifest.json b/public/manifest.json
@@ -3,7 +3,7 @@
 
   "name": "Obsidian Web",
   "description": "Connect your browser with your notes in Obsidian. Obsidian Web is a customizable web clipper for Obsidian with advanced features.",
-  "version": "3.3.3",
+  "version": "3.3.4",
 
   "options_page": "options.html",
 

diff --git a/src/popup.tsx b/src/popup.tsx
@@ -427,7 +427,7 @@ if (!document.getElementById(ROOT_CONTAINER_ID)) {
     }
 
     useEffect(() => {
-      if (popupDisplayed && !accordionIsExpanded) {
+      if (popupDisplayed && popupFormDisplayed) {
         updatePreviewContext();
         document.addEventListener(
           "keydown",
@@ -448,7 +448,7 @@ if (!document.getElementById(ROOT_CONTAINER_ID)) {
           true
         );
       };
-    }, [popupDisplayed, accordionIsExpanded, pageUrl]);
+    }, [popupDisplayed, popupFormDisplayed, pageUrl]);
 
     async function updatePreviewContext(): Promise<void> {
       setPreviewContextProcessing(true);