@richscripts/richtexteditor 2.13.0 → 2.14.0

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "2.13.0",
+  "version": "2.14.0",
   "description": "A full-featured JavaScript rich text editor (WYSIWYG) with toolbar, plugins, i18n support, a built-in AI Toolkit (Ask AI, AI Chat, AI Review drawer, BYOK resolver), structured content APIs (JSON/Markdown), and an optional collaboration / review stack: slash commands, @mentions, human-author Track Changes, threaded comments, revision history, and Yjs-backed presence. Ships first-class React, Vue, Angular and Svelte bindings, a headless \"bring your own UI\" core, and server-safe conversion helpers.",
   "main": "./richtexteditor/rte.js",
   "bin": {

package/richtexteditor/all_plugins.js

@@ -25516,20 +25516,57 @@ function RTE_Plugin_DocumentOutline() {
         var minLevel = Math.max(1, Math.min(6, config.documentOutlineMinLevel || 1));
         var maxLevel = Math.max(minLevel, Math.min(6, config.documentOutlineMaxLevel || 6));
         var headings = editable.querySelectorAll("h1,h2,h3,h4,h5,h6");
-        var counts = {};
         var items = [];
         var i;
+        // 2026-09-24 Ids must be unique in the SAVED document. This used to keep any id a
+        // heading already had, so copying or cutting a heading and pasting it saved the same
+        // id twice (measured on published 2.6.7: <h1 id="hello-world"> x2), and outline
+        // clicks / anchor links went to the first one. Now the FIRST heading in document
+        // order keeps an id and a later duplicate is renamed, so the original's links stay
+        // valid and the pasted copy becomes "slug-2". Ids on non-heading elements are
+        // reserved too, so a heading never takes one. Every heading is checked, including
+        // levels the outline does not display, since they share one id namespace.
+        var used = {};
+        var withId = editable.querySelectorAll("[id]");
+        for (i = 0; i < withId.length; i++) {
+            if (!/^H[1-6]$/.test(withId[i].nodeName)) used[withId[i].id] = true;
+        }
+        // Headings with text claim ids first. An EMPTY heading is usually the shell a cut
+        // leaves behind, still carrying the moved heading's id; letting it claim first would
+        // hand the moved heading "-2" and break every link to it. So empty headings go last
+        // and simply drop an id that a real heading already holds.
+        var textOf = function (el) {
+            return (el.innerText || el.textContent || "").replace(/\s+/g, " ").replace(/^\s+|\s+$/g, "");
+        };
+        var empties = [];
+        for (i = 0; i < headings.length; i++) {
+            var h = headings[i];
+            var t = textOf(h);
+            if (!t) { empties.push(h); continue; }
+            var base = h.id;
+            if (!base) base = slugifyHeading(t);
+            else if (!used[base]) { used[base] = true; continue; }
+            var unique = base;
+            var n = 2;
+            while (used[unique]) { unique = base + "-" + n; n++; }
+            used[unique] = true;
+            if (h.id !== unique) h.id = unique;
+        }
+        for (i = 0; i < empties.length; i++) {
+            if (!empties[i].id) continue;
+            if (used[empties[i].id]) empties[i].removeAttribute("id");
+            else used[empties[i].id] = true;
+        }
         for (i = 0; i < headings.length; i++) {
             var heading = headings[i];
             var level = parseInt(heading.nodeName.substring(1), 10);
             if (level < minLevel || level > maxLevel) continue;
             var text = (heading.innerText || heading.textContent || "").replace(/\s+/g, " ").replace(/^\s+|\s+$/g, "");
-            var baseId = heading.id || slugifyHeading(text);
-            counts[baseId] = (counts[baseId] || 0) + 1;
-            var resolvedId = heading.id || (counts[baseId] === 1 ? baseId : (baseId + "-" + counts[baseId]));
-            if (!heading.id) heading.id = resolvedId;
             items.push({
-                id: heading.id,
+                // an empty heading may now have no id; key it internally (never written to
+                // the element) so "" can't match the "no active heading" state and light up
+                // every empty entry at once.
+                id: heading.id || ("__rte-outline-" + items.length),
                 level: level,
                 text: text || ("Heading " + (items.length + 1)),
                 element: heading