autumnnote 1.6.0 → 1.6.1

package/src/js/module/VideoDialog.js

@@ -35,7 +35,7 @@ export class VideoDialog {
     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 VideoDialog {
 
   _onInsert() {
     const rawUrl = this._urlInput.value.trim();
-    const width  = Math.max(80, parseInt(this._widthInput.value, 10) || 560);
+    const width  = Math.max(80, Number.parseInt(this._widthInput.value, 10) || 560);
 
     if (!rawUrl) {
       this._urlInput.focus();
@@ -190,19 +190,19 @@ export class VideoDialog {
     } catch { return null; }
 
     // YouTube watch: https://www.youtube.com/watch?v=ID
-    const ytWatch = url.match(/(?:youtube\.com\/watch\?(?:.*&)?v=|youtube\.com\/embed\/)([a-zA-Z0-9_-]{11})/);
+    const ytWatch = /(?:youtube\.com\/watch\?(?:.*&)?v=|youtube\.com\/embed\/)([a-zA-Z0-9_-]{11})/.exec(url);
     if (ytWatch) return { type: 'YouTube', embedUrl: `https://www.youtube.com/embed/${ytWatch[1]}` };
 
     // YouTube short: https://youtu.be/ID
-    const ytShort = url.match(/youtu\.be\/([a-zA-Z0-9_-]{11})/);
+    const ytShort = /youtu\.be\/([a-zA-Z0-9_-]{11})/.exec(url);
     if (ytShort) return { type: 'YouTube', embedUrl: `https://www.youtube.com/embed/${ytShort[1]}` };
 
     // YouTube Shorts: https://www.youtube.com/shorts/ID
-    const ytShorts = url.match(/youtube\.com\/shorts\/([a-zA-Z0-9_-]{11})/);
+    const ytShorts = /youtube\.com\/shorts\/([a-zA-Z0-9_-]{11})/.exec(url);
     if (ytShorts) return { type: 'YouTube Shorts', embedUrl: `https://www.youtube.com/embed/${ytShorts[1]}` };
 
     // Vimeo: https://vimeo.com/ID
-    const vimeo = url.match(/vimeo\.com\/(\d+)/);
+    const vimeo = /vimeo\.com\/(\d+)/.exec(url);
     if (vimeo) return { type: 'Vimeo', embedUrl: `https://player.vimeo.com/video/${vimeo[1]}` };
 
     // Direct video file
@@ -238,7 +238,7 @@ export class VideoDialog {
     }
 
     if (info && info.type === 'Direct video') {
-      const src = info.embedUrl.replace(/"/g, '%22');
+      const src = info.embedUrl.replaceAll('"', '%22');
       return (
         `<div class="an-video-wrapper" style="position:relative;display:block;width:${width}px;max-width:100%">` +
         `<video src="${src}" width="${width}" height="${height}" controls ` +
@@ -258,7 +258,7 @@ export class VideoDialog {
     })();
     if (!safeSrc) return null;
 
-    const escapedSrc = safeSrc.replace(/"/g, '%22');
+    const escapedSrc = safeSrc.replaceAll('"', '%22');
     return (
       `<div class="an-video-wrapper" style="position:relative;display:block;width:${width}px;max-width:100%">` +
       `<video src="${escapedSrc}" width="${width}" height="${height}" controls ` +

package/src/js/module/VideoResizer.js

@@ -46,8 +46,8 @@ export class VideoResizer {
         if (wrapper) this._select(wrapper);
       }),
       on(document, 'click', (e) => this._onDocClick(e)),
-      on(window, 'scroll', () => this._updateOverlayPosition(), { passive: true }),
-      on(window, 'resize', onWindowResize),
+      on(globalThis, 'scroll', () => this._updateOverlayPosition(), { passive: true }),
+      on(globalThis, 'resize', onWindowResize),
       on(editable, 'scroll', () => this._updateOverlayPosition(), { passive: true }),
       // D1: Prevent native browser drag of video wrappers. Without this, a user
       // can hold-and-drag to produce a "copy" that lands outside .an-editable,
@@ -75,9 +75,7 @@ export class VideoResizer {
       this._positionRaf = null;
     }
     this._deselect();
-    if (this._overlay && this._overlay.parentNode) {
-      this._overlay.parentNode.removeChild(this._overlay);
-    }
+    this._overlay?.remove();
     this._overlay = null;
   }
 
@@ -111,7 +109,7 @@ export class VideoResizer {
   _findWrapper(el) {
     if (!el || !(el instanceof Element)) return null;
     // Direct hit on wrapper
-    if (el.classList && el.classList.contains('an-video-wrapper')) return /** @type {HTMLElement} */ (el);
+    if (el.classList?.contains('an-video-wrapper')) return /** @type {HTMLElement} */ (el);
     // Child element (iframe, video, or nested)
     const w = el.closest('.an-video-wrapper');
     if (w) return /** @type {HTMLElement} */ (w);
@@ -152,7 +150,7 @@ export class VideoResizer {
   _onDocClick(e) {
     if (!this._activeWrapper) return;
     if (this._activeWrapper.contains(e.target)) return;
-    if (this._overlay && this._overlay.contains(e.target)) return;
+    if (this._overlay?.contains(e.target)) return;
     if (e.target.closest('.an-contextmenu')) return;
     this._deselect();
   }

package/src/js/module/VideoTooltip.js

@@ -40,7 +40,7 @@ export class VideoTooltip {
         if (this.context.layoutInfo.container.classList.contains('an-disabled')) return;
         // The shield div sits on top of iframes — we detect hover via it or the wrapper
         const target = /** @type {Element} */ (e.target);
-        const wrapper = target && target.closest ? target.closest('.an-video-wrapper') : null;
+        const wrapper = target?.closest('.an-video-wrapper');
         if (wrapper && editable.contains(wrapper)) {
           this._scheduleShow(wrapper);
         }
@@ -62,8 +62,8 @@ export class VideoTooltip {
         }
       }),
       // 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;
@@ -74,7 +74,7 @@ export class VideoTooltip {
     this._clearTimers();
     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;
   }
 
@@ -210,8 +210,8 @@ export class VideoTooltip {
     let top  = rect.bottom + margin;
     let left = rect.left + (rect.width - tipW) / 2;
 
-    if (top + tipH > window.innerHeight - margin) top = rect.top - tipH - margin;
-    if (left + tipW > window.innerWidth  - margin) left = window.innerWidth - tipW - margin;
+    if (top + tipH > globalThis.innerHeight - margin) top = rect.top - tipH - margin;
+    if (left + tipW > globalThis.innerWidth  - margin) left = globalThis.innerWidth - tipW - margin;
     if (left < margin) left = margin;
 
     this._el.style.top  = `${top}px`;
@@ -268,7 +268,7 @@ export class VideoTooltip {
     if (!wrapper) return;
     this._hide();
     this.context.invoke('videoResizer.deselect');
-    if (wrapper.parentNode) wrapper.parentNode.removeChild(wrapper);
+    wrapper.remove();
     this.context.invoke('editor.afterCommand');
   }
 

package/src/js/renderer.js

@@ -46,7 +46,7 @@ export function renderLayout(targetEl, options) {
   editable.innerHTML = sanitiseHTML(initialContent, { allowIframes: true });
 
   // Apply default font family so the editable renders in the configured font
-  const defaultFont = options.defaultFontFamily || (options.fontFamilies && options.fontFamilies[0]);
+  const defaultFont = options.defaultFontFamily || options.fontFamilies?.[0];
   if (defaultFont) {
     editable.style.fontFamily = defaultFont;
   }

package/src/styles/autumnnote.scss

@@ -62,14 +62,12 @@
       // Prevent images from being dragged out.
       img {
         -webkit-user-drag: none;
-        user-drag: none;
       }
 
       // Prevent video wrapper itself from being dragged, but keep pointer-events
       // active so the video shield works and the iframe is visible/interactive.
       .an-video-wrapper {
         -webkit-user-drag: none;
-        user-drag: none;
       }
 
       // D-3: In read-only mode, disable the video shield so clicks reach the
@@ -1423,7 +1421,6 @@ $an-video-accent: #8b5cf6; // violet-500
 // .an-video-selected is applied by VideoResizer for JS state tracking.
 // Visual selection is handled entirely by the .an-video-resizer overlay,
 // so no CSS outline is needed here (two visible frames would appear otherwise).
-.an-video-selected {}
 
 // ---------------------------------------------------------------------------
 // Icon dialog