@richscripts/richtexteditor 2.5.3 → 2.6.0

package/richtexteditor/plugins/accessibilitychecker.js

@@ -408,7 +408,13 @@ function RTE_Plugin_AccessibilityChecker() {
                 if (headingText) previousHeadingLevel = level;
             }
 
-            var images = node.querySelectorAll ? node.querySelectorAll("img") : [];
+            // querySelectorAll only returns DESCENDANTS, so an image that is
+            // itself a top-level block — which is exactly where a pasted or
+            // dropped image lands in an empty editor — was never checked, and
+            // missing alt text went unreported. The table rule below already
+            // guards this case; the image rule did not. Measured 2026-08-27:
+            // bare top-level img => 0 issues, the same img inside a <p> => 1.
+            var images = tag === "img" ? [node] : (node.querySelectorAll ? node.querySelectorAll("img") : []);
             for (var imageIndex = 0; imageIndex < images.length; imageIndex++) {
                 var image = images[imageIndex];
                 if (!String(image.getAttribute("alt") || "").replace(/^\s+|\s+$/g, "")) {
@@ -425,11 +431,33 @@ function RTE_Plugin_AccessibilityChecker() {
             var tables = tag === "table" ? [node] : (node.querySelectorAll ? node.querySelectorAll("table") : []);
             for (var tableIndex = 0; tableIndex < tables.length; tableIndex++) {
                 var table = tables[tableIndex];
-                if (!table.querySelector("th")) {
+                var tableRole = String(table.getAttribute("role") || "").toLowerCase();
+                var isLayoutTable = (tableRole === "presentation" || tableRole === "none");
+
+                // 2026-08-30 A layout table MUST NOT have headers, so demanding
+                // them here produced an error the author could never clear —
+                // which is how a checker teaches people to ignore it. Presentation
+                // tables are held to the opposite rule instead: they must not
+                // carry data-table semantics, because <th>/scope/<caption> make a
+                // screen reader announce relationships that do not exist
+                // (WCAG 1.3.1). See layouttable.js.
+                if (isLayoutTable) {
+                    var semantic = table.querySelector("th, [scope], caption") || table.getAttribute("summary");
+                    if (semantic) {
+                        issues.push({
+                            code: "layout-table-semantics",
+                            severity: "error",
+                            message: "Layout table still carries data-table markup (header cells, scope or a caption). A screen reader will announce relationships this table does not have.",
+                            path: path + ".table[" + tableIndex + "]",
+                            _target: table
+                        });
+                    }
+                }
+                else if (!table.querySelector("th")) {
                     issues.push({
                         code: "table-missing-header",
                         severity: "warning",
-                        message: "Table has no header cells. Promote the first row to headers when it describes the columns.",
+                        message: "Table has no header cells. Promote the first row to headers, or mark it as a layout table when it only positions content.",
                         path: path + ".table[" + tableIndex + "]",
                         _target: table
                     });
@@ -460,7 +488,13 @@ function RTE_Plugin_AccessibilityChecker() {
             target.parentNode.replaceChild(replacement, target);
         }
         else if (issue.code === "table-missing-header") {
-            promoteFirstTableRow(target);
+            // Two valid outcomes, and only the author knows which: the table
+            // describes data (give it headers) or only positions it (say so).
+            if (options && options.markLayout) markTableAsLayout(target);
+            else promoteFirstTableRow(target);
+        }
+        else if (issue.code === "layout-table-semantics") {
+            markTableAsLayout(target);
         }
         else if (issue.code === "language-of-parts") {
             markLanguageRuns(target, options && options.lang ? options.lang : issue._lang, issue._script);
@@ -545,6 +579,35 @@ function RTE_Plugin_AccessibilityChecker() {
         }
     }
 
+    // Delegates to layouttable.js when it is loaded so there is ONE definition of
+    // what "layout table" means; the fallback keeps the repair working in builds
+    // where that plugin was left out of the bundle.
+    function markTableAsLayout(table) {
+        if (!table) return;
+        if (editor && typeof editor.setTableLayoutMode === "function") {
+            try { editor.setTableLayoutMode("layout", table); return; } catch (e) {}
+        }
+        table.setAttribute("role", "presentation");
+        table.removeAttribute("summary");
+        var caption = table.querySelector("caption");
+        if (caption && caption.parentNode === table) caption.parentNode.removeChild(caption);
+        var cells = table.querySelectorAll("th,[scope]");
+        for (var i = 0; i < cells.length; i++) {
+            var cell = cells[i];
+            // Tables nest — do not strip the headers of a data table that
+            // happens to sit inside a cell of this one.
+            if (cell.closest && cell.closest("table") !== table) continue;
+            cell.removeAttribute("scope");
+            cell.removeAttribute("headers");
+            if (String(cell.nodeName || "").toLowerCase() !== "th") continue;
+            var td = table.ownerDocument.createElement("td");
+            while (cell.firstChild) td.appendChild(cell.firstChild);
+            copyAttributes(cell, td);
+            td.removeAttribute("scope");
+            cell.parentNode.replaceChild(td, cell);
+        }
+    }
+
     function copyAttributes(source, target) {
         if (!source || !target || !source.attributes) return;
         for (var index = 0; index < source.attributes.length; index++) {

package/richtexteditor/plugins/aitoolkit.js

@@ -12,7 +12,9 @@ if (!window.RTE_DefaultConfig) window.RTE_DefaultConfig = {};
     var existing = document.querySelectorAll("link[rel=stylesheet]");
     for (var i = 0; i < existing.length; i++) {
         var href = existing[i].getAttribute("href") || "";
-        if (/(?:^|\/)aitoolkit\.css(?:\?.*)?$/i.test(href)) return;
+        // Matches the minified variant too - a host page that links either one
+        // must not get a second copy injected.
+        if (/(?:^|\/)aitoolkit(?:\.min)?\.css(?:\?.*)?$/i.test(href)) return;
     }
     // Locate our own script tag (aitoolkit.js or all_plugins.js) so we
     // can resolve aitoolkit.css next to it. Matches a custom mount path.
@@ -26,14 +28,30 @@ if (!window.RTE_DefaultConfig) window.RTE_DefaultConfig = {};
         }
     }
     var srcAttr = ourScript ? ourScript.getAttribute("src") : "";
-    var cssHref = srcAttr
-        ? srcAttr.replace(/[^/]+$/, "aitoolkit.css")
-        : (((window.RTE_DefaultConfig && window.RTE_DefaultConfig.url_base) || "/richtexteditor") + "/plugins/aitoolkit.css");
-    if (cssHref.indexOf("?") < 0) cssHref += "?v=20260703a";
+    // Prefer the minified stylesheet. aitoolkit.css is 537 KB unminified - larger
+    // than the obfuscated rte.js - and every page that loads the AI toolkit was
+    // paying for it. The readable .css is still shipped and still the one to edit
+    // or override; this only changes what gets fetched by default.
+    function resolve(name) {
+        return srcAttr
+            ? srcAttr.replace(/[^/]+$/, name)
+            : (((window.RTE_DefaultConfig && window.RTE_DefaultConfig.url_base) || "/richtexteditor") + "/plugins/" + name);
+    }
+    function bust(href) { return href.indexOf("?") < 0 ? href + "?v=20260830a" : href; }
+
     var link = document.createElement("link");
     link.rel = "stylesheet";
-    link.href = cssHref;
+    link.href = bust(resolve("aitoolkit.min.css"));
     link.setAttribute("data-rte-auto-injected", "aitoolkit");
+    // .min.css is newer than .css. An install that predates it - an unpacked older
+    // zip, a tier the mirror missed - would otherwise lose ALL AI toolkit styling
+    // silently, which is a far worse failure than shipping the larger file. Fall
+    // back to the readable stylesheet if the minified one is not there.
+    link.onerror = function () {
+        if (link.getAttribute("data-rte-css-fallback")) return;
+        link.setAttribute("data-rte-css-fallback", "1");
+        link.href = bust(resolve("aitoolkit.css"));
+    };
     (document.head || document.documentElement).appendChild(link);
 })();