Fix tray Quit and duplicate tray icon; drop idle polling and unused deps

Author: AleenaTahir1

package-lock.json

@@ -15,12 +15,9 @@
     "@tauri-apps/plugin-opener": "^2",
     "@tauri-apps/plugin-process": "^2.3.1",
     "@tauri-apps/plugin-updater": "^2.10.0",
-    "@uiw/react-md-editor": "^4.0.11",
     "framer-motion": "^12.34.0",
     "react": "^19.1.0",
     "react-dom": "^19.1.0",
-    "react-markdown": "^10.1.0",
-    "rehype-raw": "^7.0.0",
     "zustand": "^5.0.11"
   },
   "devDependencies": {

package.json

@@ -104,10 +104,14 @@ pub fn run() {
         .build(tauri::generate_context!())
         .expect("error while running tauri application")
         .run(|_app_handle, event| {
-            if let tauri::RunEvent::ExitRequested { api, .. } = event {
-                // Prevent the app from exiting when all windows are closed.
-                // The app stays alive in the system tray so global shortcuts still work.
-                api.prevent_exit();
+            if let tauri::RunEvent::ExitRequested { code, api, .. } = event {
+                // Keep the app alive in the tray ONLY when it's the windows closing
+                // (code == None) — so global shortcuts keep working. An explicit Quit
+                // from the tray calls `app.exit(0)`, which sets code == Some(0); in that
+                // case we must let the app actually exit.
+                if code.is_none() {
+                    api.prevent_exit();
+                }
             }
         });
 }

src-tauri/src/lib.rs

@@ -14,11 +14,6 @@
     "macOSPrivateApi": true,
     "security": {
       "csp": null
-    },
-    "trayIcon": {
-      "iconPath": "icons/icon.png",
-      "iconAsTemplate": true,
-      "id": "main-tray"
     }
   },
   "bundle": {

src-tauri/tauri.conf.json

@@ -47,15 +47,34 @@ export function NoteWindow({ noteId }: NoteWindowProps) {
     loadNote(noteId);
   }, [noteId, loadNote]);
 
+  // Only poll for external (Obsidian) edits when a vault is actually connected.
+  // When it isn't (the common case) nothing external changes the note, so we skip
+  // the per-window polling entirely to save CPU/battery.
+  const [syncing, setSyncing] = useState(false);
+  useEffect(() => {
+    let active = true;
+    const check = () =>
+      invoke<string | null>('get_sync_status')
+        .then((v) => active && setSyncing(!!v))
+        .catch(() => {});
+    check();
+    const id = setInterval(check, 20000); // re-check vault status occasionally (cheap)
+    return () => {
+      active = false;
+      clearInterval(id);
+    };
+  }, []);
+
   // Pick up edits made to this note inside Obsidian. Only refresh while the user is
   // NOT typing in this note's editor, so we never clobber in-progress input.
   useEffect(() => {
+    if (!syncing) return;
     const interval = setInterval(() => {
       const editingHere = document.activeElement?.classList.contains('note-editable');
       if (!editingHere) pullExternal();
-    }, 2500);
+    }, 3000);
     return () => clearInterval(interval);
-  }, [pullExternal]);
+  }, [syncing, pullExternal]);
 
   useEffect(() => {
     const handleClickOutside = () => {