autumnnote 1.6.0 → 1.6.1

package/src/js/module/LinkDialog.js

@@ -33,7 +33,7 @@ export class LinkDialog {
     this._disposers.forEach((d) => d());
     this._disposers = [];
     if (this._dialog && this._dialog.parentNode) {
-      this._dialog.parentNode.removeChild(this._dialog);
+      this._dialog.remove();
     }
     this._dialog = null;
   }
@@ -138,7 +138,7 @@ export class LinkDialog {
 
   _prefill() {
     // Try to find currently selected anchor
-    const sel = window.getSelection();
+    const sel = globalThis.getSelection();
     let anchor = null;
     if (sel && sel.rangeCount > 0) {
       let node = sel.getRangeAt(0).startContainer;

package/src/js/module/LinkTooltip.js

@@ -46,8 +46,8 @@ export class LinkTooltip {
         }
       }),
       // Hide when the page scrolls or resizes — the tooltip position becomes stale
-      on(window, 'scroll', () => this._hide(), { passive: true }),
-      on(window, 'resize', () => this._hide(), { passive: true }),
+      on(globalThis, 'scroll', () => this._hide(), { passive: true }),
+      on(globalThis, 'resize', () => this._hide(), { passive: true }),
     );
 
     return this;
@@ -57,7 +57,7 @@ export class LinkTooltip {
     this._clearTimers();
     this._disposers.forEach((d) => d());
     this._disposers = [];
-    if (this._el && this._el.parentNode) this._el.parentNode.removeChild(this._el);
+    if (this._el && this._el.parentNode) this._el.remove();
     this._el = null;
   }
 
@@ -163,11 +163,11 @@ export class LinkTooltip {
     let top  = rect.bottom + margin;
     let left = rect.left;
 
-    if (top + tipH > window.innerHeight - margin) {
+    if (top + tipH > globalThis.innerHeight - margin) {
       top = rect.top - tipH - margin;
     }
-    if (left + tipW > window.innerWidth - margin) {
-      left = window.innerWidth - tipW - margin;
+    if (left + tipW > globalThis.innerWidth - margin) {
+      left = globalThis.innerWidth - tipW - margin;
     }
     if (left < margin) left = margin;
 
@@ -190,13 +190,13 @@ export class LinkTooltip {
   // ---------------------------------------------------------------------------
 
   _openLink() {
-    const url = this._activeAnchor && this._activeAnchor.getAttribute('href');
-    if (url) window.open(url, '_blank', 'noopener,noreferrer');
+    const url = this._activeAnchor?.getAttribute('href');
+    if (url) globalThis.open(url, '_blank', 'noopener,noreferrer');
     this._hide();
   }
 
   _copyLink() {
-    const url = this._activeAnchor && this._activeAnchor.getAttribute('href');
+    const url = this._activeAnchor?.getAttribute('href');
     if (url) {
       navigator.clipboard.writeText(url).catch(() => {
         // Fallback for environments where clipboard API is unavailable
@@ -207,7 +207,7 @@ export class LinkTooltip {
         document.body.appendChild(ta);
         ta.select();
         document.execCommand('copy');
-        document.body.removeChild(ta);
+        ta.remove();
       });
     }
     // Brief visual feedback  
@@ -223,7 +223,7 @@ export class LinkTooltip {
     this._hide();
 
     // Place cursor inside the anchor, then open the link dialog (which pre-fills from selection)
-    const sel = window.getSelection();
+    const sel = globalThis.getSelection();
     const range = document.createRange();
     range.selectNodeContents(anchor);
     sel.removeAllRanges();
@@ -238,7 +238,7 @@ export class LinkTooltip {
     this._hide();
 
     // Select the anchor text, then remove the link
-    const sel = window.getSelection();
+    const sel = globalThis.getSelection();
     const range = document.createRange();
     range.selectNode(anchor);
     sel.removeAllRanges();

package/src/js/module/MarkdownShortcuts.js

@@ -61,8 +61,8 @@ export class MarkdownShortcuts {
    * @returns {{ text: string, range: Range, lineNode: Node } | null}
    */
   _getLineContext() {
-    const sel = window.getSelection();
-    if (!sel || !sel.rangeCount) return null;
+    const sel = globalThis.getSelection();
+    if (!sel?.rangeCount) return null;
     const range = sel.getRangeAt(0);
     if (!range.collapsed) return null;
 
@@ -87,7 +87,7 @@ export class MarkdownShortcuts {
 
   _isBlock(node) {
     if (node.nodeType !== Node.ELEMENT_NODE) return false;
-    const display = window.getComputedStyle(node).display;
+    const display = globalThis.getComputedStyle(node).display;
     return display === 'block' || display === 'list-item' || display === 'table-cell';
   }
 
@@ -106,7 +106,7 @@ export class MarkdownShortcuts {
     ];
 
     for (const { re, handler } of blockPatterns) {
-      const m = text.match(re);
+      const m = re.exec(text);
       if (m) {
         handler(m);
         return true;
@@ -137,8 +137,8 @@ export class MarkdownShortcuts {
   // ---------------------------------------------------------------------------
 
   _selectLineAndDelete() {
-    const sel = window.getSelection();
-    if (!sel || !sel.rangeCount) return;
+    const sel = globalThis.getSelection();
+    if (!sel?.rangeCount) return;
     const range = sel.getRangeAt(0);
     // Select from the start of the block to the cursor and delete
     const startRange = document.createRange();
@@ -188,8 +188,8 @@ export class MarkdownShortcuts {
   // ---------------------------------------------------------------------------
 
   _onInput() {
-    const sel = window.getSelection();
-    if (!sel || !sel.rangeCount) return;
+    const sel = globalThis.getSelection();
+    if (!sel?.rangeCount) return;
     const range = sel.getRangeAt(0);
     if (!range.collapsed) return;
 
@@ -215,7 +215,7 @@ export class MarkdownShortcuts {
 
     const upToCursor = text.slice(0, offset);
     for (const { re, tag } of inlineRules) {
-      const m = upToCursor.match(re);
+      const m = re.exec(upToCursor);
       if (!m) continue;
 
       const matchStart = upToCursor.length - m[0].length;
@@ -233,11 +233,8 @@ export class MarkdownShortcuts {
       const beforeNode = document.createTextNode(before);
       const afterNode = document.createTextNode('​' + after);
 
-      const parent = node.parentNode;
-      parent.insertBefore(beforeNode, node);
-      parent.insertBefore(el, node);
-      parent.insertBefore(afterNode, node);
-      parent.removeChild(node);
+      /** @type {ChildNode} */ (node).before(beforeNode, el, afterNode);
+      /** @type {ChildNode} */ (node).remove();
 
       // Place cursor after the element (after the ZWS)
       const newRange = document.createRange();

package/src/js/module/Mention.js

@@ -76,9 +76,7 @@ export class Mention {
 
   destroy() {
     clearTimeout(this._debounceTimer);
-    if (this._dropdown && this._dropdown.parentNode) {
-      this._dropdown.parentNode.removeChild(this._dropdown);
-    }
+    this._dropdown?.remove();
     this._dropdown = null;
     this._disposers.forEach((d) => d());
     this._disposers = [];
@@ -163,10 +161,10 @@ export class Mention {
     let top = rect.bottom + 4;
     let left = rect.left;
 
-    if (rect.bottom + ddh + 8 > window.innerHeight) {
+    if (rect.bottom + ddh + 8 > globalThis.innerHeight) {
       top = rect.top - ddh - 4;
     }
-    left = Math.max(8, Math.min(left, window.innerWidth - ddw - 8));
+    left = Math.max(8, Math.min(left, globalThis.innerWidth - ddw - 8));
 
     dd.style.top = `${top}px`;
     dd.style.left = `${left}px`;
@@ -196,7 +194,7 @@ export class Mention {
   _captureCaretRect() {
     // Prefer a range over the trigger character — it has non-zero width and
     // getBoundingClientRect() reliably returns a valid rect.
-    if (this._triggerNode && this._triggerNode.isConnected) {
+    if (this._triggerNode?.isConnected) {
       try {
         const r = document.createRange();
         const end = Math.min(this._triggerOffset + 1, this._triggerNode.textContent.length);
@@ -211,7 +209,7 @@ export class Mention {
     }
 
     // Fallback: use getClientRects() on the current collapsed selection
-    const sel = window.getSelection();
+    const sel = globalThis.getSelection();
     if (!sel || !sel.rangeCount) return;
     const rects = sel.getRangeAt(0).getClientRects();
     if (rects.length > 0) {
@@ -224,7 +222,7 @@ export class Mention {
   // ---------------------------------------------------------------------------
 
   _getQueryAtCursor() {
-    const sel = window.getSelection();
+    const sel = globalThis.getSelection();
     if (!sel || !sel.rangeCount) return null;
     const range = sel.getRangeAt(0);
     if (!range.collapsed) return null;
@@ -305,7 +303,7 @@ export class Mention {
 
   _onDocClick(e) {
     if (!this._open) return;
-    if (this._dropdown && this._dropdown.contains(e.target)) return;
+    if (this._dropdown?.contains(e.target)) return;
     this._hideDropdown();
   }
 
@@ -325,7 +323,7 @@ export class Mention {
       node.textContent = before + after;
 
       // Move cursor to where the trigger was
-      const sel = window.getSelection();
+      const sel = globalThis.getSelection();
       const range = document.createRange();
       range.setStart(node, this._triggerOffset);
       range.collapse(true);

package/src/js/module/Placeholder.js

@@ -43,7 +43,7 @@ export class Placeholder {
     // addition to regular whitespace before deciding if the editor is empty.
     // Without this, a freshly-created checklist item or icon leaves a ZWS in
     // the DOM that causes the placeholder to overlap real content (A-1).
-    const hasText = editable.textContent.replace(/\u200B/g, '').trim().length > 0;
+    const hasText = editable.textContent.replaceAll('\u200B', '').trim().length > 0;
     const isEmpty = !hasText &&
       !editable.querySelector('img, table, hr, .an-video-wrapper');
     editable.classList.toggle('an-placeholder', isEmpty && !isFocused);

package/src/js/module/ShortcutsDialog.js

@@ -69,9 +69,7 @@ export class ShortcutsDialog {
   destroy() {
     this._disposers.forEach((d) => d());
     this._disposers = [];
-    if (this._dialog && this._dialog.parentNode) {
-      this._dialog.parentNode.removeChild(this._dialog);
-    }
+    this._dialog?.remove();
     this._dialog = null;
   }
 
@@ -79,7 +77,7 @@ export class ShortcutsDialog {
     if (this._dialog) {
       this._dialog.style.display = 'flex';
       this._removeTrap = trapFocus(this._dialog, () => this._close());
-      setTimeout(() => this._closeBtn && this._closeBtn.focus(), 50);
+      setTimeout(() => this._closeBtn?.focus(), 50);
     }
   }
 

package/src/js/module/Statusbar.js

@@ -109,9 +109,7 @@ export class Statusbar {
       this._dragDisposers.forEach((d) => d());
       this._dragDisposers = null;
     }
-    if (this.el && this.el.parentNode) {
-      this.el.parentNode.removeChild(this.el);
-    }
+    this.el?.remove();
     this.el = null;
   }
 
@@ -217,7 +215,7 @@ export class Statusbar {
     // textContent is faster than innerText (no layout flush, no CSS visibility check)
     const text = editable.textContent || '';
     const words = _countWords(text);
-    const chars = text.replace(/\n/g, '').length;
+    const chars = text.replaceAll('\n', '').length;
     const maxWords = this.options.maxWords || 0;
     const maxChars = this.options.maxChars || 0;
 
@@ -249,6 +247,6 @@ export class Statusbar {
    */
   getCharCount() {
     const editable = this.context.layoutInfo.editable;
-    return ((editable.innerText || '').replace(/\n/g, '')).length;
+    return (editable.innerText || '').replaceAll('\n', '').length;
   }
 }

package/src/js/module/TableTooltip.js

@@ -212,26 +212,26 @@ export class TableTooltip {
         if (!to || (
           !editable.contains(to) &&
           !this._el.contains(to) &&
-          !(this._sizePopover && this._sizePopover.contains(to))
+          !this._sizePopover?.contains(to)
         )) {
           this._scheduleHide();
         }
       }, { passive: true }),
       on(document, 'click', (e) => {
         const et = /** @type {Node} */ (e.target);
-        if (this._selectMode && this._activeTable && this._activeTable.contains(et)) return;
+        if (this._selectMode && this._activeTable?.contains(et)) return;
         if (this._activeTable &&
           !this._activeTable.contains(et) &&
           !this._el.contains(et) &&
-          !(this._sizePopover && this._sizePopover.contains(et))) {
+          !this._sizePopover?.contains(et)) {
           this._hide();
         }
       }),
       // Sync shade strip whenever selection moves to a different cell
       on(document, 'selectionchange', () => this._syncShadeStrip()),
       // Hide when the page scrolls or resizes — the tooltip position becomes stale
-      on(window, 'scroll',  () => this._hide(), { passive: true }),
-      on(window, 'resize',  () => this._hide(), { passive: true }),
+      on(globalThis, 'scroll',  () => this._hide(), { passive: true }),
+      on(globalThis, 'resize',  () => this._hide(), { passive: true }),
     );
 
     this._initResize();
@@ -386,14 +386,14 @@ export class TableTooltip {
     this._clearTimers();
     this._disposers.forEach((d) => d());
     this._disposers = [];
-    if (this._el && this._el.parentNode) this._el.parentNode.removeChild(this._el);
+    if (this._el && this._el.parentNode) this._el.remove();
     this._el = null;
     if (this._sizePopover && this._sizePopover.parentNode) {
-      this._sizePopover.parentNode.removeChild(this._sizePopover);
+      this._sizePopover.remove();
     }
     this._sizePopover = null;
     if (this._shadePopover && this._shadePopover.parentNode) {
-      this._shadePopover.parentNode.removeChild(this._shadePopover);
+      this._shadePopover.remove();
     }
     this._shadePopover = null;
   }
@@ -553,7 +553,7 @@ export class TableTooltip {
   _syncShadeStrip() {
     if (!this._shadeColorStrip || !this._el || this._el.style.display === 'none') return;
     const cell = this._getCell();
-    this._shadeColorStrip.style.background = (cell && cell.style.backgroundColor) || 'transparent';
+    this._shadeColorStrip.style.background = cell?.style.backgroundColor || 'transparent';
   }
 
   _hide() {
@@ -590,7 +590,7 @@ export class TableTooltip {
     let top  = rect.top - tipH - margin;
 
     if (top  < margin)                           top  = rect.bottom + margin;
-    if (left + tipW > window.innerWidth - margin) left = window.innerWidth - tipW - margin;
+    if (left + tipW > globalThis.innerWidth - margin) left = globalThis.innerWidth - tipW - margin;
     if (left < margin)                            left = margin;
 
     this._el.style.left = `${left}px`;
@@ -603,17 +603,17 @@ export class TableTooltip {
 
   _getCell() {
     // Prefer the cell under the current text cursor (most intuitive for operations)
-    const sel = window.getSelection();
+    const sel = globalThis.getSelection();
     if (sel && sel.rangeCount) {
       let container = sel.getRangeAt(0).commonAncestorContainer;
       if (container.nodeType === 3) container = container.parentElement;
-      const cellFromSel = container && /** @type {Element} */ (container).closest('td, th');
-      if (cellFromSel && this._activeTable && this._activeTable.contains(cellFromSel)) {
+      const cellFromSel = /** @type {Element} */ (container)?.closest('td, th');
+      if (cellFromSel && this._activeTable?.contains(cellFromSel)) {
         return cellFromSel;
       }
     }
     return this._activeCell
-      || (this._activeTable && this._activeTable.querySelector('td, th'));
+      || this._activeTable?.querySelector('td, th');
   }
 
   // ---------------------------------------------------------------------------
@@ -735,7 +735,7 @@ export class TableTooltip {
       const bi = allRows.indexOf(best);
       const ri = allRows.indexOf(r);
       return position === 'above' ? (ri < bi ? r : best) : (ri > bi ? r : best);
-    });
+    }, selectedRows[0]);
     const colCount = Array.from(refRow.cells).reduce((sum, c) => sum + (c.colSpan || 1), 0);
     const newRow = document.createElement('tr');
     const refCells = Array.from(refRow.cells);
@@ -786,7 +786,7 @@ export class TableTooltip {
     if (bodyRowsToDelete.length >= totalBodyRows) return;
     this._activeCell = null;
     this._clearSelection();
-    selectedRows.forEach((r) => r.parentElement?.removeChild(r));
+    selectedRows.forEach((r) => r.remove());
     requestAnimationFrame(() => this._positionNear(this._activeTable));
     this.context.invoke('editor.afterCommand');
   }
@@ -810,7 +810,7 @@ export class TableTooltip {
     });
     this._activeCell = null;
     this._clearSelection();
-    cellsToDelete.forEach((c) => c.parentElement?.removeChild(c));
+    cellsToDelete.forEach((c) => c.remove());
     requestAnimationFrame(() => this._positionNear(this._activeTable));
     this.context.invoke('editor.afterCommand');
   }
@@ -824,7 +824,7 @@ export class TableTooltip {
     // Prefer user panel-selected cells; fall back to text-selection range
     let selected = this._getSelectedCells().filter((c) => table.contains(c));
     if (selected.length < 2) {
-      const sel = window.getSelection();
+      const sel = globalThis.getSelection();
       if (!sel || sel.rangeCount === 0) return;
       const range = sel.getRangeAt(0);
       const allCells = Array.from(table.querySelectorAll('td, th'));
@@ -866,7 +866,7 @@ export class TableTooltip {
     first.rowSpan = maxR - minR + 1;
     first.style.verticalAlign = 'middle';
     first.innerHTML = rectCells.map((c) => c.innerHTML).join('');
-    rectCells.slice(1).forEach((c) => c.parentElement?.removeChild(c));
+    rectCells.slice(1).forEach((c) => c.remove());
 
     this._clearSelection();
     this.context.invoke('editor.afterCommand');
@@ -876,7 +876,7 @@ export class TableTooltip {
     const table = this._activeTable;
     if (!table) return;
     this._hide();
-    if (table.parentNode) table.parentNode.removeChild(table);
+    if (table.parentNode) table.remove();
     this.context.invoke('editor.afterCommand');
   }
 
@@ -981,7 +981,7 @@ export class TableTooltip {
     this._sizeApply   = null;
 
     const d1 = on(applyBtn, 'click', () => {
-      const val = parseInt(this._sizeInputEl.value, 10);
+      const val = Number.parseInt(this._sizeInputEl.value, 10);
       if (val > 0 && typeof this._sizeApply === 'function') this._sizeApply(val);
       this._hideSizePopover();
     });
@@ -1016,8 +1016,8 @@ export class TableTooltip {
       if (!table) return;
       const firstCell = table.querySelector('td, th');
       const currentPx = firstCell
-        ? (parseInt(firstCell.style.borderWidth, 10) ||
-           parseInt(window.getComputedStyle(firstCell).borderWidth, 10) || 1)
+        ? (Number.parseInt(firstCell.style.borderWidth, 10) ||
+           Number.parseInt(globalThis.getComputedStyle(firstCell).borderWidth, 10) || 1)
         : 1;
       this._sizeTitleEl.textContent = this.context.locale.tooltips.table.tableBorderWidthPx;
       this._sizeInputEl.min   = '0';
@@ -1079,8 +1079,8 @@ export class TableTooltip {
       const ph = this._sizePopover.offsetHeight || 110;
       let left = tipRect.left;
       let top  = tipRect.bottom + 6;
-      if (left + pw > window.innerWidth  - 8) left = window.innerWidth  - pw - 8;
-      if (top  + ph > window.innerHeight - 8) top  = tipRect.top - ph - 6;
+      if (left + pw > globalThis.innerWidth  - 8) left = globalThis.innerWidth  - pw - 8;
+      if (top  + ph > globalThis.innerHeight - 8) top  = tipRect.top - ph - 6;
       this._sizePopover.style.left = `${left}px`;
       this._sizePopover.style.top  = `${top}px`;
       if (this._sizeInputEl) { this._sizeInputEl.focus(); this._sizeInputEl.select(); }
@@ -1135,11 +1135,9 @@ export class TableTooltip {
     customRow.appendChild(customLabel);
     pop.appendChild(customRow);
 
-    // Prevent mousedown from collapsing editor selection
-    this._disposers.push(on(pop, 'mousedown', (e) => e.preventDefault()));
-
-    // Keep tooltip alive while hovering popover
+    // Prevent mousedown from collapsing editor selection; keep tooltip alive while hovering
     this._disposers.push(
+      on(pop, 'mousedown', (e) => e.preventDefault()),
       on(pop, 'mouseenter', () => this._clearTimers()),
       on(pop, 'mouseleave', () => this._scheduleHide()),
     );
@@ -1150,7 +1148,7 @@ export class TableTooltip {
       if (this._shadePopover &&
           this._shadePopover.style.display !== 'none' &&
           !this._shadePopover.contains(et) &&
-          !(this._el && this._el.contains(et))) {
+          !this._el?.contains(et)) {
         this._hideCellShadePopover();
       }
     }));
@@ -1172,8 +1170,8 @@ export class TableTooltip {
       const tipRect = this._el.getBoundingClientRect();
       let left = tipRect.left;
       let top  = tipRect.bottom + 6;
-      if (left + pw > window.innerWidth  - 8) left = window.innerWidth  - pw - 8;
-      if (top  + ph > window.innerHeight - 8) top  = tipRect.top - ph - 6;
+      if (left + pw > globalThis.innerWidth  - 8) left = globalThis.innerWidth  - pw - 8;
+      if (top  + ph > globalThis.innerHeight - 8) top  = tipRect.top - ph - 6;
       this._shadePopover.style.left = `${Math.max(8, left)}px`;
       this._shadePopover.style.top  = `${Math.max(8, top)}px`;
     });

package/src/js/module/Toolbar.js

@@ -146,7 +146,7 @@ export class Toolbar {
     this._disposers.forEach((d) => d());
     this._disposers = [];
     if (this.el && this.el.parentNode) {
-      this.el.parentNode.removeChild(this.el);
+      this.el.remove();
     }
     this.el = null;
   }
@@ -244,8 +244,8 @@ export class Toolbar {
 
     const setHighlight = (rows, cols) => {
       cells.forEach((cell) => {
-        const r = +cell.getAttribute('data-row');
-        const c = +cell.getAttribute('data-col');
+        const r = +cell.dataset.row;
+        const c = +cell.dataset.col;
         cell.classList.toggle('active', r <= rows && c <= cols);
       });
       label.textContent = (rows && cols) ? `${rows} × ${cols}` : (this.context.locale.toolbar.insertTableLabel || 'Insert Table');
@@ -264,8 +264,8 @@ export class Toolbar {
 
       let left = rect.left;
       let top  = rect.bottom + 4;
-      if (left + pw > window.innerWidth - 8) left = Math.max(8, window.innerWidth - pw - 8);
-      if (top  + ph > window.innerHeight - 8) top  = rect.top - ph - 4;
+      if (left + pw > globalThis.innerWidth - 8) left = Math.max(8, globalThis.innerWidth - pw - 8);
+      if (top  + ph > globalThis.innerHeight - 8) top  = rect.top - ph - 4;
 
       popup.style.left = `${left}px`;
       popup.style.top  = `${top}px`;
@@ -286,18 +286,18 @@ export class Toolbar {
     });
 
     const d2 = on(grid, 'mouseover', (e) => {
-      const cell = /** @type {Element} */ (e.target)?.closest('.an-table-cell');
+      const cell = /** @type {HTMLElement|null} */ (/** @type {Element} */ (e.target)?.closest('.an-table-cell'));
       if (!cell) return;
-      setHighlight(+cell.getAttribute('data-row'), +cell.getAttribute('data-col'));
+      setHighlight(+cell.dataset.row, +cell.dataset.col);
     });
 
     const d3 = on(grid, 'mouseleave', () => setHighlight(0, 0));
 
     const d4 = on(grid, 'click', (e) => {
-      const cell = /** @type {Element} */ (e.target)?.closest('.an-table-cell');
+      const cell = /** @type {HTMLElement|null} */ (/** @type {Element} */ (e.target)?.closest('.an-table-cell'));
       if (!cell) return;
-      const rows = +cell.getAttribute('data-row');
-      const cols = +cell.getAttribute('data-col');
+      const rows = +cell.dataset.row;
+      const cols = +cell.dataset.col;
       closePopup();
       this.context.invoke('editor.focus');
       def.action(this.context, rows, cols);
@@ -308,7 +308,7 @@ export class Toolbar {
     // Append popup to body so position:fixed is truly viewport-relative,
     // unaffected by any ancestor transform / filter (same pattern as color picker).
     this._disposers.push(d1, d2, d3, d4, d5, () => {
-      if (popup.parentNode) popup.parentNode.removeChild(popup);
+      if (popup.parentNode) popup.remove();
     });
 
     wrap.appendChild(btn);
@@ -398,14 +398,14 @@ export class Toolbar {
     let savedRange = null;
 
     const saveSelection = () => {
-      const sel = window.getSelection();
+      const sel = globalThis.getSelection();
       savedRange = (sel && sel.rangeCount) ? sel.getRangeAt(0).cloneRange() : null;
     };
 
     const restoreSelection = () => {
       if (!savedRange) return;
       try {
-        const sel = window.getSelection();
+        const sel = globalThis.getSelection();
         if (!sel) return;
         sel.removeAllRanges();
         sel.addRange(savedRange);
@@ -424,7 +424,7 @@ export class Toolbar {
       const rect = arrowBtn.getBoundingClientRect();
       const popupMinW = 184;
       let left = rect.left;
-      if (left + popupMinW > window.innerWidth) left = rect.right - popupMinW;
+      if (left + popupMinW > globalThis.innerWidth) left = rect.right - popupMinW;
       popup.style.top  = `${rect.bottom + 4}px`;
       popup.style.left = `${Math.max(4, left)}px`;
       popup.style.display = 'block';
@@ -491,13 +491,13 @@ export class Toolbar {
     // popup doesn't drift away from the button it belongs to.
     const onScrollResize = () => { if (isOpen) closePopup(); };
     document.addEventListener('scroll', onScrollResize, { passive: true, capture: true });
-    window.addEventListener('resize',   onScrollResize, { passive: true });
+    globalThis.addEventListener('resize',   onScrollResize, { passive: true });
 
     this._disposers.push(d1, d2, d2b, d3, d3b, d4, d5, d6,
       () => document.removeEventListener('scroll', onScrollResize, { capture: true }),
-      () => window.removeEventListener('resize',   onScrollResize),
+      () => globalThis.removeEventListener('resize',   onScrollResize),
       // Remove popup from body on editor destroy
-      () => { if (popup.parentNode) popup.parentNode.removeChild(popup); },
+      () => { if (popup.parentNode) popup.remove(); },
     );
 
     // Register this popup's closer so other color pickers can close it
@@ -562,7 +562,7 @@ export class Toolbar {
     /** @type {Range|null} */
     let _savedRange = null;
     const dMousedown = on(select, 'mousedown', () => {
-      const sel = window.getSelection();
+      const sel = globalThis.getSelection();
       _savedRange = (sel && sel.rangeCount) ? sel.getRangeAt(0).cloneRange() : null;
     });
 
@@ -574,7 +574,7 @@ export class Toolbar {
       // Restore selection saved on mousedown so the action targets the correct text.
       if (_savedRange) {
         try {
-          const sel = window.getSelection();
+          const sel = globalThis.getSelection();
           if (sel) { sel.removeAllRanges(); sel.addRange(_savedRange); }
         } catch (_) { /* range may be stale if DOM changed */ }
       }
@@ -684,7 +684,7 @@ export class Toolbar {
 
     // Sync button active states
     this.el.querySelectorAll('button[data-btn]').forEach((btn) => {
-      const def = btnMap.get(btn.getAttribute('data-btn'));
+      const def = btnMap.get(/** @type {HTMLElement} */ (btn).dataset.btn);
       if (def && typeof def.isActive === 'function') {
         btn.classList.toggle('active', !!def.isActive(this.context));
       }
@@ -695,7 +695,7 @@ export class Toolbar {
 
     // Sync select dropdowns (e.g. font family) with current cursor position
     this.el.querySelectorAll('select[data-btn]').forEach((select) => {
-      const def = btnMap.get(select.getAttribute('data-btn'));
+      const def = btnMap.get(/** @type {HTMLElement} */ (select).dataset.btn);
       if (!def || typeof def.getValue !== 'function') return;
       // queryCommandValue returns the font name, possibly quoted — strip quotes
       let raw = (def.getValue(this.context) || '').replace(/["']/g, '').trim();