feat(tree): disable animation for performance

apps/client/src/components/app_context.ts

@@ -541,7 +541,6 @@ export type FilteredCommandNames<T extends CommandData> = keyof Pick<CommandMapp
 
 export class AppContext extends Component {
     isMainWindow: boolean;
-    windowId: string;
     components: Component[];
     beforeUnloadListeners: (WeakRef<BeforeUploadListener> | (() => boolean))[];
     tabManager!: TabManager;
@@ -550,11 +549,10 @@ export class AppContext extends Component {
 
     lastSearchString?: string;
 
-    constructor(isMainWindow: boolean, windowId: string) {
+    constructor(isMainWindow: boolean) {
         super();
 
         this.isMainWindow = isMainWindow;
-        this.windowId = windowId;
         // non-widget/layout components needed for the application
         this.components = [];
         this.beforeUnloadListeners = [];
@@ -684,7 +682,8 @@ export class AppContext extends Component {
         this.beforeUnloadListeners = this.beforeUnloadListeners.filter(l => l !== listener);
     }
 }
-const appContext = new AppContext(window.glob.isMainWindow, window.glob.windowId);
+
+const appContext = new AppContext(window.glob.isMainWindow);
 
 // we should save all outstanding changes before the page/app is closed
 $(window).on("beforeunload", () => {

apps/client/src/components/entrypoints.ts

@@ -142,15 +142,14 @@ export default class Entrypoints extends Component {
     }
 
     async openInWindowCommand({ notePath, hoistedNoteId, viewScope }: NoteCommandData) {
-        const extraWindowId = utils.randomString(4);
         const extraWindowHash = linkService.calculateHash({ notePath, hoistedNoteId, viewScope });
 
         if (utils.isElectron()) {
             const { ipcRenderer } = utils.dynamicRequire("electron");
 
-            ipcRenderer.send("create-extra-window", { extraWindowId, extraWindowHash });
+            ipcRenderer.send("create-extra-window", { extraWindowHash });
         } else {
-            const url = `${window.location.protocol}//${window.location.host}${window.location.pathname}?extraWindow=${extraWindowId}${extraWindowHash}`;
+            const url = `${window.location.protocol}//${window.location.host}${window.location.pathname}?extraWindow=1${extraWindowHash}`;
 
             window.open(url, "", "width=1000,height=800");
         }

apps/client/src/components/tab_manager.ts

@@ -11,8 +11,6 @@ import linkService from "../services/link.js";
 import type { EventData } from "./app_context.js";
 import type FNote from "../entities/fnote.js";
 
-const MAX_SAVED_WINDOWS = 10;
-
 interface TabState {
     contexts: NoteContext[];
     position: number;
@@ -27,13 +25,6 @@ interface NoteContextState {
     viewScope: Record<string, any>;
 }
 
-interface WindowState {
-    windowId: string;
-    createdAt: number;
-    closedAt: number;
-    contexts: NoteContextState[];
-}
-
 export default class TabManager extends Component {
     public children: NoteContext[];
     public mutex: Mutex;
@@ -50,6 +41,9 @@ export default class TabManager extends Component {
         this.recentlyClosedTabs = [];
 
         this.tabsUpdate = new SpacedUpdate(async () => {
+            if (!appContext.isMainWindow) {
+                return;
+            }
             if (options.is("databaseReadonly")) {
                 return;
             }
@@ -58,21 +52,9 @@ export default class TabManager extends Component {
                 .map((nc) => nc.getPojoState())
                 .filter((t) => !!t);
 
-            // Update the current window’s openNoteContexts in options    
-            const savedWindows = options.getJson("openNoteContexts") || [];
-            const win = savedWindows.find((w: WindowState) => w.windowId === appContext.windowId);
-            if (win) {
-                win.contexts = openNoteContexts;
-            } else {
-                savedWindows.push({
-                    windowId: appContext.windowId,
-                    createdAt: Date.now(),
-                    closedAt: 0,
-                    contexts: openNoteContexts
-                } as WindowState);
-            }
-
-            await options.save("openNoteContexts", JSON.stringify(savedWindows));
+            await server.put("options", {
+                openNoteContexts: JSON.stringify(openNoteContexts)
+            });
         });
 
         appContext.addBeforeUnloadListener(this);
@@ -87,13 +69,8 @@ export default class TabManager extends Component {
     }
 
     async loadTabs() {
-        // Get the current window’s openNoteContexts
-        const savedWindows = options.getJson("openNoteContexts") || [];
-        const currentWin = savedWindows.find(w => w.windowId === appContext.windowId);
-        const openNoteContexts = currentWin ? currentWin.contexts : undefined;
-
         try {
-            const noteContextsToOpen = openNoteContexts || [];
+            const noteContextsToOpen = (appContext.isMainWindow && options.getJson("openNoteContexts")) || [];
 
             // preload all notes at once
             await froca.getNotes([...noteContextsToOpen.flatMap((tab: NoteContextState) =>
@@ -142,51 +119,6 @@ export default class TabManager extends Component {
                 }
             });
 
-            // Save window contents
-            if (currentWin as WindowState) {
-                currentWin.createdAt = Date.now();
-                currentWin.closedAt = 0;
-                currentWin.contexts = filteredNoteContexts;
-            } else {
-                if (savedWindows?.length >= MAX_SAVED_WINDOWS) {
-                    // Filter out the oldest entry
-                    // 1) Never remove the "main" window
-                    // 2) Prefer removing the oldest closed window (closedAt !== 0)
-                    // 3) If no closed window exists, remove the window with the oldest created window
-                    let oldestClosedIndex = -1;
-                    let oldestClosedTime = Infinity;
-                    let oldestCreatedIndex = -1;
-                    let oldestCreatedTime = Infinity;
-                    savedWindows.forEach((w: WindowState, i: number) => {
-                        if (w.windowId === "main") return;
-                        if (w.closedAt !== 0) {
-                            if (w.closedAt < oldestClosedTime) {
-                                oldestClosedTime = w.closedAt;
-                                oldestClosedIndex = i;
-                            }
-                        } else {
-                            if (w.createdAt < oldestCreatedTime) {
-                                oldestCreatedTime = w.createdAt;
-                                oldestCreatedIndex = i;
-                            }
-                        }
-                    });
-                    const indexToRemove = oldestClosedIndex !== -1 ? oldestClosedIndex : oldestCreatedIndex;
-                    if (indexToRemove !== -1) {
-                        savedWindows.splice(indexToRemove, 1);
-                    }
-                }
-
-                savedWindows.push({
-                    windowId: appContext.windowId,
-                    createdAt: Date.now(),
-                    closedAt: 0,
-                    contexts: filteredNoteContexts
-                } as WindowState);
-            }
-
-            await options.save("openNoteContexts", JSON.stringify(savedWindows));
-
             // if there's a notePath in the URL, make sure it's open and active
             // (useful, for e.g., opening clipped notes from clipper or opening link in an extra window)
             if (parsedFromUrl.notePath) {

apps/client/src/services/froca_updater.ts

@@ -27,6 +27,10 @@ async function processEntityChanges(entityChanges: EntityChange[]) {
                 loadResults.addRevision(ec.entityId, ec.noteId, ec.componentId);
             } else if (ec.entityName === "options") {
                 const attributeEntity = ec.entity as FAttributeRow;
+                if (attributeEntity.name === "openNoteContexts") {
+                    continue; // only noise
+                }
+
                 options.set(attributeEntity.name as OptionNames, attributeEntity.value);
                 loadResults.addOption(attributeEntity.name as OptionNames);
             } else if (ec.entityName === "attachments") {

apps/client/src/test/setup.ts

@@ -13,8 +13,7 @@ function injectGlobals() {
     uncheckedWindow.$ = $;
     uncheckedWindow.WebSocket = () => {};
     uncheckedWindow.glob = {
-        isMainWindow: true,
-        windowId: "main"
+        isMainWindow: true
     };
 }
 

apps/client/src/types.d.ts

@@ -36,7 +36,6 @@ interface CustomGlobals {
     isProtectedSessionAvailable: boolean;
     isDev: boolean;
     isMainWindow: boolean;
-    windowId: string;
     maxEntityChangeIdAtLoad: number;
     maxEntityChangeSyncIdAtLoad: number;
     assetPath: string;

apps/client/src/widgets/layout/StatusBar.tsx

@@ -5,7 +5,7 @@ import { Dropdown as BootstrapDropdown } from "bootstrap";
 import clsx from "clsx";
 import { type ComponentChildren, RefObject } from "preact";
 import { createPortal } from "preact/compat";
-import { useContext, useEffect, useMemo, useRef, useState } from "preact/hooks";
+import { useCallback, useContext, useEffect, useMemo, useRef, useState } from "preact/hooks";
 
 import { CommandNames } from "../../components/app_context";
 import NoteContext from "../../components/note_context";
@@ -338,15 +338,19 @@ interface AttributesProps extends StatusBarContext {
 function AttributesButton({ note, attributesShown, setAttributesShown }: AttributesProps) {
     const [ count, setCount ] = useState(note.attributes.length);
 
+    const refreshCount = useCallback((note: FNote) => {
+        return note.getAttributes().filter(a => !a.isAutoLink).length;
+    }, []);
+
     // React to note changes.
     useEffect(() => {
-        setCount(note.getAttributes().filter(a => !a.isAutoLink).length);
-    }, [ note ]);
+        setCount(refreshCount(note));
+    }, [ note, refreshCount ]);
 
     // React to changes in count.
     useTriliumEvent("entitiesReloaded", (({loadResults}) => {
         if (loadResults.getAttributeRows().some(attr => attributes.isAffecting(attr, note))) {
-            setCount(note.attributes.length);
+            setCount(refreshCount(note));
         }
     }));
 

apps/client/src/widgets/note_tree.ts

@@ -153,7 +153,7 @@ const TPL = /*html*/`
 const MAX_SEARCH_RESULTS_IN_TREE = 100;
 
 // this has to be hanged on the actual elements to effectively intercept and stop click event
-const cancelClickPropagation: (e: JQuery.ClickEvent) => void = (e) => e.stopPropagation();
+const cancelClickPropagation: (e: JQuery.ClickEvent | MouseEvent) => void = (e) => e.stopPropagation();
 
 // TODO: Fix once we remove Node.js API from public
 type Timeout = NodeJS.Timeout | string | number | undefined;
@@ -353,6 +353,7 @@ export default class NoteTreeWidget extends NoteContextAwareWidget {
         this.$tree.fancytree({
             titlesTabbable: true,
             keyboard: true,
+            toggleEffect: false,
             extensions: ["dnd5", "clones", "filter"],
             source: treeData,
             scrollOfs: {
@@ -598,102 +599,7 @@ export default class NoteTreeWidget extends NoteContextAwareWidget {
             clones: {
                 highlightActiveClones: true
             },
-            async enhanceTitle (
-                event: Event,
-                data: {
-                    node: Fancytree.FancytreeNode;
-                    noteId: string;
-                }
-            ) {
-                const node = data.node;
-
-                if (!node.data.noteId) {
-                    // if there's "non-note" node, then don't enhance
-                    // this can happen for e.g. "Load error!" node
-                    return;
-                }
-
-                const note = await froca.getNote(node.data.noteId, true);
-
-                if (!note) {
-                    return;
-                }
-
-                const activeNoteContext = appContext.tabManager.getActiveContext();
-
-                const $span = $(node.span);
-
-                $span.find(".tree-item-button").remove();
-                $span.find(".note-indicator-icon").remove();
-
-                const isHoistedNote = activeNoteContext && activeNoteContext.hoistedNoteId === note.noteId && note.noteId !== "root";
-
-                if (note.hasLabel("workspace") && !isHoistedNote) {
-                    const $enterWorkspaceButton = $(`<span class="tree-item-button tn-icon enter-workspace-button bx bx-door-open" title="${t("note_tree.hoist-this-note-workspace")}"></span>`).on(
-                        "click",
-                        cancelClickPropagation
-                    );
-
-                    $span.append($enterWorkspaceButton);
-                }
-
-                if (note.type === "search") {
-                    const $refreshSearchButton = $(`<span class="tree-item-button tn-icon refresh-search-button bx bx-refresh" title="${t("note_tree.refresh-saved-search-results")}"></span>`).on(
-                        "click",
-                        cancelClickPropagation
-                    );
-
-                    $span.append($refreshSearchButton);
-                }
-
-                // TODO: Deduplicate with server's notes.ts#getAndValidateParent
-                if (!["search", "launcher"].includes(note.type)
-                    && !note.isOptions()
-                    && !note.isLaunchBarConfig()
-                    && !note.noteId.startsWith("_help")
-                ) {
-                    const $createChildNoteButton = $(`<span class="tree-item-button tn-icon add-note-button bx bx-plus" title="${t("note_tree.create-child-note")}"></span>`).on(
-                        "click",
-                        cancelClickPropagation
-                    );
-
-                    $span.append($createChildNoteButton);
-                }
-
-                if (isHoistedNote) {
-                    const $unhoistButton = $(`<span class="tree-item-button tn-icon unhoist-button bx bx-door-open" title="${t("note_tree.unhoist")}"></span>`).on("click", cancelClickPropagation);
-
-                    $span.append($unhoistButton);
-                }
-
-                // Add clone indicator with tooltip if note has multiple parents
-                const parentNotes = note.getParentNotes();
-                const realParents = parentNotes.filter(
-                    (parent) => !["_share", "_lbBookmarks"].includes(parent.noteId) && parent.type !== "search"
-                );
-
-                if (realParents.length > 1) {
-                    const parentTitles = realParents.map((p) => p.title).join(", ");
-                    const tooltipText = realParents.length === 2
-                        ? t("note_tree.clone-indicator-tooltip-single", { parent: realParents[1].title })
-                        : t("note_tree.clone-indicator-tooltip", { count: realParents.length, parents: parentTitles });
-
-                    const $cloneIndicator = $(`<span class="note-indicator-icon clone-indicator"></span>`);
-                    $cloneIndicator.attr("title", tooltipText);
-                    $span.find(".fancytree-title").append($cloneIndicator);
-                }
-
-                // Add shared indicator with tooltip if note is shared
-                if (note.isShared()) {
-                    const shareId = note.getOwnedLabelValue("shareAlias") || note.noteId;
-                    const shareUrl = `${location.origin}${location.pathname}share/${shareId}`;
-                    const tooltipText = t("note_tree.shared-indicator-tooltip-with-url", { url: shareUrl });
-
-                    const $sharedIndicator = $(`<span class="note-indicator-icon shared-indicator"></span>`);
-                    $sharedIndicator.attr("title", tooltipText);
-                    $span.find(".fancytree-title").append($sharedIndicator);
-                }
-            },
+            enhanceTitle: buildEnhanceTitle(),
             // this is done to automatically lazy load all expanded notes after tree load
             loadChildren: (event, data) => {
                 data.node.visit((subNode) => {
@@ -1882,3 +1788,100 @@ export default class NoteTreeWidget extends NoteContextAwareWidget {
         return items;
     }
 }
+
+function buildEnhanceTitle() {
+    const createChildTemplate = document.createElement("span");
+    createChildTemplate.className = "tree-item-button tn-icon add-note-button bx bx-plus";
+    createChildTemplate.title = t("note_tree.create-child-note");
+    createChildTemplate.addEventListener("click", cancelClickPropagation);
+
+    return async function enhanceTitle(event: Event,
+        data: {
+            node: Fancytree.FancytreeNode;
+            noteId: string;
+        }) {
+        const node = data.node;
+
+        if (!node.data.noteId) {
+            // if there's "non-note" node, then don't enhance
+            // this can happen for e.g. "Load error!" node
+            return;
+        }
+
+        const note = froca.getNoteFromCache(node.data.noteId);
+        if (!note) return;
+
+        const activeNoteContext = appContext.tabManager.getActiveContext();
+
+        const $span = $(node.span);
+
+        $span.find(".tree-item-button").remove();
+        $span.find(".note-indicator-icon").remove();
+
+        const isHoistedNote = activeNoteContext && activeNoteContext.hoistedNoteId === note.noteId && note.noteId !== "root";
+
+        if (note.hasLabel("workspace") && !isHoistedNote) {
+            const $enterWorkspaceButton = $(`<span class="tree-item-button tn-icon enter-workspace-button bx bx-door-open" title="${t("note_tree.hoist-this-note-workspace")}"></span>`).on(
+                "click",
+                cancelClickPropagation
+            );
+
+            $span.append($enterWorkspaceButton);
+        }
+
+        if (note.type === "search") {
+            const $refreshSearchButton = $(`<span class="tree-item-button tn-icon refresh-search-button bx bx-refresh" title="${t("note_tree.refresh-saved-search-results")}"></span>`).on(
+                "click",
+                cancelClickPropagation
+            );
+
+            $span.append($refreshSearchButton);
+        }
+
+        // TODO: Deduplicate with server's notes.ts#getAndValidateParent
+        if (!["search", "launcher"].includes(note.type)
+            && !note.isOptions()
+            && !note.isLaunchBarConfig()
+            && !note.noteId.startsWith("_help")
+        ) {
+            node.span.append(createChildTemplate.cloneNode());
+        }
+
+        if (isHoistedNote) {
+            const $unhoistButton = $(`<span class="tree-item-button tn-icon unhoist-button bx bx-door-open" title="${t("note_tree.unhoist")}"></span>`).on("click", cancelClickPropagation);
+
+            $span.append($unhoistButton);
+        }
+
+        // Add clone indicator with tooltip if note has multiple parents
+        const parentNotes = note.getParentNotes();
+        const realParents: FNote[] = [];
+        for (const parent of parentNotes) {
+            if (parent.noteId !== "_share" && parent.noteId !== "_lbBookmarks" && parent.type !== "search") {
+                realParents.push(parent);
+            }
+        }
+
+        if (realParents.length > 1) {
+            const parentTitles = realParents.map((p) => p.title).join(", ");
+            const tooltipText = realParents.length === 2
+                ? t("note_tree.clone-indicator-tooltip-single", { parent: realParents[1].title })
+                : t("note_tree.clone-indicator-tooltip", { count: realParents.length, parents: parentTitles });
+
+            const $cloneIndicator = $(`<span class="note-indicator-icon clone-indicator"></span>`);
+            $cloneIndicator.attr("title", tooltipText);
+            $span.find(".fancytree-title").append($cloneIndicator);
+        }
+
+        // Add shared indicator with tooltip if note is shared
+        if (note.isShared()) {
+            const shareId = note.getOwnedLabelValue("shareAlias") || note.noteId;
+            const shareUrl = `${location.origin}${location.pathname}share/${shareId}`;
+            const tooltipText = t("note_tree.shared-indicator-tooltip-with-url", { url: shareUrl });
+
+            const $sharedIndicator = $(`<span class="note-indicator-icon shared-indicator"></span>`);
+            $sharedIndicator.attr("title", tooltipText);
+            $span.find(".fancytree-title").append($sharedIndicator);
+        }
+    };
+}

apps/desktop/src/main.ts

@@ -6,7 +6,6 @@ import sqlInit from "@triliumnext/server/src/services/sql_init.js";
 import windowService from "@triliumnext/server/src/services/window.js";
 import tray from "@triliumnext/server/src/services/tray.js";
 import options from "@triliumnext/server/src/services/options.js";
-
 import electronDebug from "electron-debug";
 import electronDl from "electron-dl";
 import { PRODUCT_NAME } from "./app-info";
@@ -70,12 +69,10 @@ async function main() {
         globalShortcut.unregisterAll();
     });
 
-    app.on("second-instance", async (event, commandLine) => {
+    app.on("second-instance", (event, commandLine) => {
         const lastFocusedWindow = windowService.getLastFocusedWindow();
         if (commandLine.includes("--new-window")) {
-            const randomString = (await import("@triliumnext/server/src/services/utils.js")).randomString;
-            const extraWindowId = randomString(4);
-            windowService.createExtraWindow(extraWindowId, "");
+            windowService.createExtraWindow("");
         } else if (lastFocusedWindow) {
             if (lastFocusedWindow.isMinimized()) {
                 lastFocusedWindow.restore();
@@ -127,8 +124,7 @@ async function onReady() {
                 }
             });
         }
-        
-        await normalizeOpenNoteContexts();
+
         tray.createTray();
     } else {
         await windowService.createSetupWindow();
@@ -137,30 +133,6 @@ async function onReady() {
     await windowService.registerGlobalShortcuts();
 }
 
-/**
- * Some windows may have closed abnormally, leaving closedAt as 0 in openNoteContexts.
- * This function normalizes those timestamps to the current time for correct sorting/filtering.
- */
-async function normalizeOpenNoteContexts() {
-    const savedWindows = options.getOptionJson("openNoteContexts") || [];
-    const now = Date.now();
-
-    let changed = false;
-    for (const win of savedWindows) {
-        if (win.windowId !== "main" && win.closedAt === 0) {
-            win.closedAt = now;
-            changed = true;
-        }
-    }
-    
-    if (changed) {
-        const { default: cls } = (await import("@triliumnext/server/src/services/cls.js"));
-        cls.wrap(() => {
-            options.setOption("openNoteContexts", JSON.stringify(savedWindows));
-        })();
-    }
-}
-
 function getElectronLocale() {
     const uiLocale = options.getOptionOrNull("locale");
     const formattingLocale = options.getOptionOrNull("formattingLocale");

apps/server-e2e/src/note_types/text.spec.ts

@@ -42,7 +42,7 @@ test("Highlights list is displayed", async ({ page, context }) => {
     await app.closeAllTabs();
     await app.goToNoteInNewTab("Highlights list");
 
-    await expect(app.sidebar).toContainText(/highlights/i);
+    await expect(app.sidebar).toContainText("Highlights List");
     const rootList = app.sidebar.locator(".highlights-list ol");
     let index = 0;
     for (const highlightedEl of ["Bold 1", "Italic 1", "Underline 1", "Colored text 1", "Background text 1", "Bold 2", "Italic 2", "Underline 2", "Colored text 2", "Background text 2"]) {

apps/server-e2e/src/support/app.ts

@@ -59,7 +59,7 @@ export default class App {
 
         // Wait for the page to load.
         if (url === "/") {
-            await expect(this.noteTree).toContainText("Trilium Integration Test");
+            await expect(this.page.locator(".tree")).toContainText("Trilium Integration Test");
             if (!preserveTabs) {
                 await this.closeAllTabs();
             }

apps/server/src/assets/translations/en/server.json

@@ -382,8 +382,6 @@
     "tooltip": "Trilium Notes",
     "close": "Quit Trilium",
     "recents": "Recent notes",
-    "recently-closed-windows": "Recently closed windows",
-    "tabs-total": "{{number}} tabs total",
     "bookmarks": "Bookmarks",
     "today": "Open today's journal note",
     "new-note": "New note",

apps/server/src/assets/views/desktop.ejs

@@ -15,7 +15,7 @@
 </head>
 <body
   id="trilium-app"
-  class="desktop heading-style-<%= headingStyle %> layout-<%= layoutOrientation %> platform-<%= platform %> <%= isElectron ? 'electron' : '' %> <%= hasNativeTitleBar ? 'native-titlebar' : '' %> <%= hasBackgroundEffects ? 'background-effects' : '' %> <%= isMainWindow ? '' : 'extra-window' %>"
+  class="desktop heading-style-<%= headingStyle %> layout-<%= layoutOrientation %> platform-<%= platform %> <%= isElectron ? 'electron' : '' %> <%= hasNativeTitleBar ? 'native-titlebar' : '' %> <%= hasBackgroundEffects ? 'background-effects' : '' %>"
   lang="<%= currentLocale.id %>" dir="<%= currentLocale.rtl ? 'rtl' : 'ltr' %>"
 >
 <noscript><%= t("javascript-required") %></noscript>

apps/server/src/assets/views/partials/windowGlobal.ejs

@@ -12,7 +12,6 @@
         isDev: <%= isDev %>,
         appCssNoteIds: <%- JSON.stringify(appCssNoteIds) %>,
         isMainWindow: <%= isMainWindow %>,
-        windowId: "<%= windowId %>",
         isProtectedSessionAvailable: <%= isProtectedSessionAvailable %>,
         triliumVersion: "<%= triliumVersion %>",
         assetPath: "<%= assetPath %>",

apps/server/src/migrations/0234__migrate_open_note_contexts_format.ts

@@ -1,48 +0,0 @@
-import cls from "../services/cls.js";
-import sql from "../services/sql.js";
-
-export default () => {
-    cls.init(() => {
-        const row = sql.getRow<{ value: string }>(
-            `SELECT value FROM options WHERE name = 'openNoteContexts'`
-        );
-
-        if (!row || !row.value) {
-            return;
-        }
-
-        let parsed: any;
-        try {
-            parsed = JSON.parse(row.value);
-        } catch {
-            return;
-        }
-
-        // Already in new format (array + windowId), skip
-        if (
-            Array.isArray(parsed) &&
-            parsed.length > 0 &&
-            parsed[0] &&
-            typeof parsed[0] === "object" &&
-            parsed[0].windowId
-        ) {
-            return;
-        }
-
-        // Old format: just contexts
-        const migrated = [
-            {
-                windowId: "main",
-                createdAt: 0,
-                closedAt: 0,
-                contexts: parsed
-            }
-        ];
-
-        sql.execute(
-            `UPDATE options SET value = ? WHERE name = 'openNoteContexts'`,
-            [JSON.stringify(migrated)]
-        );
-
-    });
-};

apps/server/src/migrations/migrations.ts

@@ -6,11 +6,6 @@
 
 // Migrations should be kept in descending order, so the latest migration is first.
 const MIGRATIONS: (SqlMigration | JsMigration)[] = [
-    // Migrate openNoteContexts option to the new structured format with window metadata
-    {
-        version: 234,
-        module: async () => import("./0234__migrate_open_note_contexts_format")
-    },
     // Migrate geo map to collection
     {
         version: 233,

apps/server/src/routes/index.ts

@@ -56,7 +56,6 @@ function index(req: Request, res: Response) {
         appCssNoteIds: getAppCssNoteIds(),
         isDev,
         isMainWindow: view === "mobile" ? true : !req.query.extraWindow,
-        windowId: view !== "mobile" && req.query.extraWindow ? req.query.extraWindow : "main",
         isProtectedSessionAvailable: protectedSessionService.isProtectedSessionAvailable(),
         triliumVersion: packageJson.version,
         assetPath,

apps/server/src/services/app_info.ts

@@ -4,7 +4,7 @@ import packageJson from "../../package.json" with { type: "json" };
 import dataDir from "./data_dir.js";
 import { AppInfo } from "@triliumnext/commons";
 
-const APP_DB_VERSION = 234;
+const APP_DB_VERSION = 233;
 const SYNC_VERSION = 36;
 const CLIPPER_PROTOCOL_VERSION = "1.0";
 

apps/server/src/services/options.ts

@@ -72,19 +72,6 @@ function getOptionBool(name: FilterOptionsByType<boolean>): boolean {
     return val === "true";
 }
 
-function getOptionJson(name: OptionNames) {
-    const val = getOptionOrNull(name);
-
-    if (typeof val !== "string") {
-        return null;
-    }
-    try {
-        return JSON.parse(val);
-    } catch (e) {
-        return null;
-    }
-}
-
 function setOption<T extends OptionNames>(name: T, value: string | OptionDefinitions[T]) {
     const option = becca.getOption(name);
 
@@ -150,7 +137,6 @@ export default {
     getOption,
     getOptionInt,
     getOptionBool,
-    getOptionJson,
     setOption,
     createOption,
     getOptions,

apps/server/src/services/options_init.ts

@@ -45,15 +45,8 @@ async function initNotSyncedOptions(initialized: boolean, opts: NotSyncedOpts =
         "openNoteContexts",
         JSON.stringify([
             {
-                windowId: "main",
-                createdAt: 0,
-                closedAt: 0,
-                contexts: [
-                    {
-                        notePath: "root",
-                        active: true
-                    }
-                ]
+                notePath: "root",
+                active: true
             }
         ]),
         false
@@ -264,15 +257,8 @@ function initStartupOptions() {
             "openNoteContexts",
             JSON.stringify([
                 {
-                    windowId: "main",
-                    createdAt: 0,
-                    closedAt: 0,
-                    contexts: [
-                        {
-                            notePath: process.env.TRILIUM_START_NOTE_ID || "root",
-                            active: true
-                        }
-                    ]
+                    notePath: process.env.TRILIUM_START_NOTE_ID || "root",
+                    active: true
                 }
             ])
         );

apps/server/src/services/sql_init.ts

@@ -147,15 +147,8 @@ async function createInitialDatabase(skipDemoDb?: boolean) {
             "openNoteContexts",
             JSON.stringify([
                 {
-                    windowId: "main",
-                    createdAt: 0,
-                    closedAt: 0,
-                    contexts: [
-                        {
-                            notePath: startNoteId,
-                            active: true
-                        }
-                    ]
+                    notePath: startNoteId,
+                    active: true
                 }
             ])
         );

apps/server/src/services/tray.ts

@@ -196,39 +196,6 @@ function updateTrayMenu() {
         return menuItems;
     }
 
-    function buildClosedWindowsMenu() {
-        const savedWindows = optionService.getOptionJson("openNoteContexts") || [];
-        const openedWindowIds = windowService.getAllWindowIds();
-        const closedWindows = savedWindows
-            .filter(win => !openedWindowIds.includes(win.windowId))
-            .sort((a, b) => { return a.closedAt - b.closedAt; });  // sort by time in ascending order
-        const menuItems: Electron.MenuItemConstructorOptions[] = [];
-        
-        for (let i = closedWindows.length - 1; i >= 0; i--) {
-            const win = closedWindows[i];
-            const activeCtx = win.contexts.find(c => c.active === true);
-            const activateNotePath = (activeCtx ?? win.contexts[0])?.notePath;
-            const activateNoteId = activateNotePath?.split("/").pop() ?? null;
-            if (!activateNoteId) continue;
-            
-            // Get the title of the closed window
-            const winTitle = (() => {
-                const raw = becca_service.getNoteTitle(activateNoteId);
-                const truncated = raw.length > 20 ? `${raw.slice(0, 17)}…` : raw;
-                const tabCount = win.contexts.filter(ctx => ctx.mainNtxId === null).length;
-                return tabCount > 1 ? `${truncated} (${t("tray.tabs-total", { number: tabCount })})` : truncated;
-            })();
-
-            menuItems.push({
-                label: winTitle,
-                type: "normal",
-                click: () => win.windowId !== "main" ? windowService.createExtraWindow(win.windowId, "") : windowService.createMainWindow()
-            });
-        }
-
-        return menuItems;
-    }
-
     const windowVisibilityMenuItems: Electron.MenuItemConstructorOptions[] = [];
 
     // Only call getWindowTitle if windowVisibilityMap has more than one window
@@ -291,12 +258,6 @@ function updateTrayMenu() {
             icon: getIconPath("recents"),
             submenu: buildRecentNotesMenu()
         },
-        {
-            label: t("tray.recently-closed-windows"),
-            type: "submenu",
-            icon: getIconPath("closed-windows"),
-            submenu: buildClosedWindowsMenu()
-        },
         { type: "separator" },
         {
             label: t("tray.close"),

apps/server/src/services/window.ts

@@ -16,45 +16,28 @@ import { formatDownloadTitle, isMac, isWindows } from "./utils.js";
 // Prevent the window being garbage collected
 let mainWindow: BrowserWindow | null;
 let setupWindow: BrowserWindow | null;
+let allWindows: BrowserWindow[] = []; // // Used to store all windows, sorted by the order of focus.
 
-interface WindowEntry {
-    window: BrowserWindow;
-    windowId: string; // custom window ID
-}
-let allWindowEntries: WindowEntry[] = [];
-
-function trackWindowFocus(win: BrowserWindow, windowId: string) {
+function trackWindowFocus(win: BrowserWindow) {
     // We need to get the last focused window from allWindows. If the last window is closed, we return the previous window.
     // Therefore, we need to push the window into the allWindows array every time it gets focused.
     win.on("focus", () => {
-        allWindowEntries = allWindowEntries.filter(w => !w.window.isDestroyed() && w.window !== win);
-        allWindowEntries.push({ window: win, windowId: windowId });
-
+        allWindows = allWindows.filter(w => !w.isDestroyed() && w !== win);
+        allWindows.push(win);
         if (!optionService.getOptionBool("disableTray")) {
             electron.ipcMain.emit("reload-tray");
         }
     });
 
     win.on("closed", () => {
-        cls.wrap(() => {
-            const savedWindows = optionService.getOptionJson("openNoteContexts") || [];
-
-            const win = savedWindows.find(w => w.windowId === windowId);
-            if (win) {
-                win.closedAt = Date.now();
-            }
-
-            optionService.setOption("openNoteContexts", JSON.stringify(savedWindows));
-        })();
-        
-        allWindowEntries = allWindowEntries.filter(w => !w.window.isDestroyed());
+        allWindows = allWindows.filter(w => !w.isDestroyed());
         if (!optionService.getOptionBool("disableTray")) {
             electron.ipcMain.emit("reload-tray");
         }
     });
 }
 
-async function createExtraWindow(extraWindowId: string, extraWindowHash: string) {
+async function createExtraWindow(extraWindowHash: string) {
     const spellcheckEnabled = optionService.getOptionBool("spellCheckEnabled");
 
     const { BrowserWindow } = await import("electron");
@@ -73,15 +56,15 @@ async function createExtraWindow(extraWindowId: string, extraWindowHash: string)
     });
 
     win.setMenuBarVisibility(false);
-    win.loadURL(`http://127.0.0.1:${port}/?extraWindow=${extraWindowId}${extraWindowHash}`);
+    win.loadURL(`http://127.0.0.1:${port}/?extraWindow=1${extraWindowHash}`);
 
     configureWebContents(win.webContents, spellcheckEnabled);
 
-    trackWindowFocus(win, extraWindowId);
+    trackWindowFocus(win);
 }
 
 electron.ipcMain.on("create-extra-window", (event, arg) => {
-    createExtraWindow(arg.extraWindowId, arg.extraWindowHash);
+    createExtraWindow(arg.extraWindowHash);
 });
 
 interface PrintOpts {
@@ -185,8 +168,8 @@ async function getBrowserWindowForPrinting(e: IpcMainEvent, notePath: string, ac
     return { browserWindow, printReport };
 }
 
-async function createMainWindow(app?: App) {
-    if (app && "setUserTasks" in app) {
+async function createMainWindow(app: App) {
+    if ("setUserTasks" in app) {
         app.setUserTasks([
             {
                 program: process.execPath,
@@ -236,7 +219,7 @@ async function createMainWindow(app?: App) {
     mainWindow.on("closed", () => (mainWindow = null));
 
     configureWebContents(mainWindow.webContents, spellcheckEnabled);
-    trackWindowFocus(mainWindow, "main");
+    trackWindowFocus(mainWindow);
 }
 
 function getWindowExtraOpts() {
@@ -398,15 +381,11 @@ function getMainWindow() {
 }
 
 function getLastFocusedWindow() {
-    return allWindowEntries.length > 0 ? allWindowEntries[allWindowEntries.length - 1]?.window : null;
+    return allWindows.length > 0 ? allWindows[allWindows.length - 1] : null;
 }
 
 function getAllWindows() {
-    return allWindowEntries.map(e => e.window);
-}
-
-function getAllWindowIds(): string[] {
-    return allWindowEntries.map(e => e.windowId);
+    return allWindows;
 }
 
 export default {
@@ -417,6 +396,5 @@ export default {
     registerGlobalShortcuts,
     getMainWindow,
     getLastFocusedWindow,
-    getAllWindows,
-    getAllWindowIds
+    getAllWindows
 };

packages/ckeditor5/src/i18n.ts

@@ -50,6 +50,11 @@ const LOCALE_MAPPINGS: Record<DISPLAYABLE_LOCALE_IDS, LocaleMapping | null> = {
         coreTranslation: () => import("ckeditor5/translations/ja.js"),
         premiumFeaturesTranslation: () => import("ckeditor5-premium-features/translations/ja.js"),
     },
+    pl: {
+        languageCode: "pl",
+        coreTranslation: () => import("ckeditor5/translations/pl.js"),
+        premiumFeaturesTranslation: () => import("ckeditor5-premium-features/translations/pl.js"),
+    },
     pt: {
         languageCode: "pt",
         coreTranslation: () => import("ckeditor5/translations/pt.js"),