@elementor/editor-audits 4.4.0-1129 → 4.4.0-1131

package/dist/index.mjs

@@ -208,16 +208,16 @@ async function requestPageContext(documentId, imageSizeRequests, allowNonceRetry
 
 // src/registry.ts
 var registry = /* @__PURE__ */ new Map();
-function registerAudit(audit26) {
-  registry.set(audit26.id, audit26);
+function registerAudit(audit25) {
+  registry.set(audit25.id, audit25);
 }
 function getRegisteredAudits() {
   return Array.from(registry.values());
 }
 
 // src/utils/is-scored-audit.ts
-function isScoredAudit(audit26) {
-  return audit26.weight > 0;
+function isScoredAudit(audit25) {
+  return audit25.weight > 0;
 }
 
 // src/utils/sort-failed-audits.ts
@@ -278,16 +278,16 @@ function computeReport(documentId, results) {
   const accumulators = Object.fromEntries(
     ALL_CATEGORIES.map((c) => [c, { totalWeight: 0, passedWeight: 0, total: 0, failed: 0 }])
   );
-  for (const { audit: audit26, result } of results) {
+  for (const { audit: audit25, result } of results) {
     if (result.status === "skipped") {
       continue;
     }
-    for (const category of audit26.categories) {
+    for (const category of audit25.categories) {
       const acc = accumulators[category];
       acc.total++;
-      acc.totalWeight += audit26.weight;
+      acc.totalWeight += audit25.weight;
       if (result.status === "pass") {
-        acc.passedWeight += audit26.weight;
+        acc.passedWeight += audit25.weight;
       } else {
         acc.failed++;
       }
@@ -364,13 +364,14 @@ function recurse(nodes, parents, visit) {
 }
 
 // src/utils/atomic-image-sources.ts
-var ATOMIC_IMAGE_PROP_KEYS = ["image", "poster"];
-function walkAtomicImageSources(tree, visit) {
+var ATOMIC_CONTENT_IMAGE_PROP_KEYS = ["image"];
+var ATOMIC_IMAGE_PROP_KEYS = [...ATOMIC_CONTENT_IMAGE_PROP_KEYS, "poster"];
+function walkAtomicImageSources(tree, visit, propKeys = ATOMIC_IMAGE_PROP_KEYS) {
   walkElements(tree, (node) => {
     if (node.elType !== "widget") {
       return;
     }
-    for (const key of ATOMIC_IMAGE_PROP_KEYS) {
+    for (const key of propKeys) {
       const media = extractAtomicImageMedia(node.settings[key]);
       if (media) {
         visit({ node, media });
@@ -392,8 +393,9 @@ function extractAtomicImageMedia(imageProp) {
   if (!id && !url) {
     return null;
   }
+  const alt = stringPropTypeUtil.extract(src.alt) ?? void 0;
   const size = stringPropTypeUtil.extract(image.size) ?? void 0;
-  return { id, url, size };
+  return { id, url, alt, size };
 }
 
 // src/utils/atomic-background-image-sources.ts
@@ -448,15 +450,6 @@ function walkBackgroundImageSources(tree, visit) {
 
 // src/utils/image-like-sources.ts
 var IMAGE_LIKE_WIDGETS = /* @__PURE__ */ new Set(["image", "image-box", "image-carousel", "gallery", "image-gallery"]);
-function hasPageImages(tree) {
-  let found = false;
-  walkImageLikeSources(tree, ({ media }) => {
-    if (media.id || media.url) {
-      found = true;
-    }
-  });
-  return found;
-}
 function walkImageLikeSources(tree, visit) {
   walkElements(tree, (node) => {
     if (node.elType !== "widget") {
@@ -488,6 +481,14 @@ function collectMediaFromNode(node) {
   return sources;
 }
 
+// src/utils/all-image-sources.ts
+function walkAllImageSources(tree, visit) {
+  walkImageLikeSources(tree, visit);
+  walkBackgroundImageSources(tree, visit);
+  walkAtomicImageSources(tree, visit);
+  walkAtomicBackgroundImageSources(tree, visit);
+}
+
 // src/utils/image-size-key.ts
 function buildImageSizeKey({ id, size }) {
   return `${id}:${size || "full"}`;
@@ -496,15 +497,11 @@ function buildImageSizeKey({ id, size }) {
 // src/utils/page-attachments.ts
 function extractImageSizeRequests(tree) {
   const keys = /* @__PURE__ */ new Set();
-  const collect = (media) => {
+  walkAllImageSources(tree, ({ media }) => {
     if (media.id) {
       keys.add(buildImageSizeKey({ id: media.id, size: media.size }));
     }
-  };
-  walkImageLikeSources(tree, ({ media }) => collect(media));
-  walkBackgroundImageSources(tree, ({ media }) => collect(media));
-  walkAtomicImageSources(tree, ({ media }) => collect(media));
-  walkAtomicBackgroundImageSources(tree, ({ media }) => collect(media));
+  });
   return Array.from(keys).sort();
 }
 
@@ -613,10 +610,10 @@ async function runPageAudit(documentId) {
   const ctx = { documentId, elements, pageContext, kit, renderedHtml };
   const registered = getRegisteredAudits();
   const auditResults = await Promise.all(
-    registered.map(async (audit26) => {
-      const { evaluate: _evaluate, ...meta } = audit26;
+    registered.map(async (audit25) => {
+      const { evaluate: _evaluate, ...meta } = audit25;
       try {
-        const result = await audit26.evaluate(ctx);
+        const result = await audit25.evaluate(ctx);
         return { audit: meta, result };
       } catch (error) {
         const reason = error instanceof Error ? error.message : "unknown-error";
@@ -1109,9 +1106,9 @@ function ViolationIcon({ violation, widgetIcon }) {
 function SkipReasonTooltip({ reason }) {
   return /* @__PURE__ */ React12.createElement(Tooltip3, { title: reason, placement: "top" }, /* @__PURE__ */ React12.createElement(Box7, { "aria-label": reason, component: "span", sx: { display: "inline-flex", alignItems: "center" } }, /* @__PURE__ */ React12.createElement(HelpIcon, { fontSize: "small", color: "action" })));
 }
-function StatusIndicator({ audit: audit26, violations }) {
+function StatusIndicator({ audit: audit25, violations }) {
   if (violations) {
-    return /* @__PURE__ */ React12.createElement(React12.Fragment, null, /* @__PURE__ */ React12.createElement(SeverityIcon, { severity: audit26.severity }), isScoredAudit(audit26) && /* @__PURE__ */ React12.createElement(Typography6, { variant: "caption", color: "text.secondary", fontWeight: "bold" }, violations.length));
+    return /* @__PURE__ */ React12.createElement(React12.Fragment, null, /* @__PURE__ */ React12.createElement(SeverityIcon, { severity: audit25.severity }), isScoredAudit(audit25) && /* @__PURE__ */ React12.createElement(Typography6, { variant: "caption", color: "text.secondary", fontWeight: "bold" }, violations.length));
   }
   return /* @__PURE__ */ React12.createElement(CheckIcon, { fontSize: "small", color: "success" });
 }
@@ -1153,7 +1150,7 @@ function getSharedAngiePrompt(violations) {
   );
   return allShareTheSameFix ? first.angiePrompt : void 0;
 }
-function ViolationRow({ audit: audit26, expanded, onToggleExpand, skipReason, violations }) {
+function ViolationRow({ audit: audit25, expanded, onToggleExpand, skipReason, violations }) {
   const primaryViolation = violations?.[0];
   const isGuidanceFocusable = !!primaryViolation;
   const panelZIndex = useFloatingPanelZIndex2(AUDIT_PANEL_ID);
@@ -1176,8 +1173,8 @@ function ViolationRow({ audit: audit26, expanded, onToggleExpand, skipReason, vi
       },
       onClick: onToggleExpand
     },
-    /* @__PURE__ */ React12.createElement(Typography6, { variant: "subtitle2", color: "text.secondary", sx: { flex: 1 } }, audit26.title),
-    !skipReason && /* @__PURE__ */ React12.createElement(StatusIndicator, { audit: audit26, violations })
+    /* @__PURE__ */ React12.createElement(Typography6, { variant: "subtitle2", color: "text.secondary", sx: { flex: 1 } }, audit25.title),
+    !skipReason && /* @__PURE__ */ React12.createElement(StatusIndicator, { audit: audit25, violations })
   ), skipReason && /* @__PURE__ */ React12.createElement(SkipReasonTooltip, { reason: skipReason }), /* @__PURE__ */ React12.createElement(
     IconButton5,
     {
@@ -1195,7 +1192,7 @@ function ViolationRow({ audit: audit26, expanded, onToggleExpand, skipReason, vi
         }
       }
     )
-  )), /* @__PURE__ */ React12.createElement(Collapse2, { in: expanded }, /* @__PURE__ */ React12.createElement(Box7, { paddingBlock: 1 }, /* @__PURE__ */ React12.createElement(Tooltip3, { title: audit26.fixHint, placement: "top" }, /* @__PURE__ */ React12.createElement(
+  )), /* @__PURE__ */ React12.createElement(Collapse2, { in: expanded }, /* @__PURE__ */ React12.createElement(Box7, { paddingBlock: 1 }, /* @__PURE__ */ React12.createElement(Tooltip3, { title: audit25.fixHint, placement: "top" }, /* @__PURE__ */ React12.createElement(
     Alert,
     {
       severity: "secondary",
@@ -1206,7 +1203,7 @@ function ViolationRow({ audit: audit26, expanded, onToggleExpand, skipReason, vi
       onClick: isGuidanceFocusable ? handleGuidanceClick : void 0,
       onKeyDown: isGuidanceFocusable ? onKeyboardClick(handleGuidanceClick) : void 0
     },
-    /* @__PURE__ */ React12.createElement(Typography6, { variant: "caption", component: "p", color: "text.secondary" }, audit26.description),
+    /* @__PURE__ */ React12.createElement(Typography6, { variant: "caption", component: "p", color: "text.secondary" }, audit25.description),
     primaryViolation && /* @__PURE__ */ React12.createElement(
       GuidanceAction,
       {
@@ -1324,17 +1321,17 @@ import { __ as __12 } from "@wordpress/i18n";
 var ALL_SEVERITIES = ["error", "warning", "info"];
 function countSeverities(report, category) {
   const counts = { error: 0, warning: 0, info: 0 };
-  for (const { audit: audit26, result } of report.auditResults) {
+  for (const { audit: audit25, result } of report.auditResults) {
     if (result.status !== "fail") {
       continue;
     }
-    if (category && !audit26.categories.includes(category)) {
+    if (category && !audit25.categories.includes(category)) {
       continue;
     }
-    if (!isScoredAudit(audit26)) {
+    if (!isScoredAudit(audit25)) {
       continue;
     }
-    counts[audit26.severity] += result.violations.length;
+    counts[audit25.severity] += result.violations.length;
   }
   return counts;
 }
@@ -1540,7 +1537,7 @@ function SeverityCard({ severity, count }) {
       flex: 1,
       flexDirection: "column",
       gap: 1,
-      bgcolor: "grey.50",
+      bgcolor: "action.hover",
       borderRadius: 1.5,
       px: 1.5,
       py: 1
@@ -2277,6 +2274,21 @@ var audit10 = {
 // src/audits/images-alt-text.ts
 import { __ as __29 } from "@wordpress/i18n";
 
+// src/utils/alt-text-image-sources.ts
+function walkAltTextImageSources(tree, visit) {
+  walkImageLikeSources(tree, visit);
+  walkAtomicImageSources(tree, visit, ATOMIC_CONTENT_IMAGE_PROP_KEYS);
+}
+function hasAltTextImages(tree) {
+  let found = false;
+  walkAltTextImageSources(tree, ({ media }) => {
+    if (media.id || media.url) {
+      found = true;
+    }
+  });
+  return found;
+}
+
 // src/utils/image-alt.ts
 function hasMeaningfulAlt(media, pageContext) {
   if (!media.id && !media.url) {
@@ -2303,14 +2315,14 @@ var audit11 = {
   severity: "error",
   weight: 10,
   evaluate: (ctx) => {
-    if (!hasPageImages(ctx.elements.tree)) {
+    if (!hasAltTextImages(ctx.elements.tree)) {
       return { status: "skipped", reason: __29("No images", "elementor") };
     }
     const isReady = ctx.pageContext.ally_plugin_active;
     const violations = [];
     const failedWidgetIds = /* @__PURE__ */ new Set();
     let missingAltImageCount = 0;
-    walkImageLikeSources(ctx.elements.tree, ({ node, media }) => {
+    walkAltTextImageSources(ctx.elements.tree, ({ node, media }) => {
       if (!isImageSourcePresent(media) || hasMeaningfulAlt(media, ctx.pageContext)) {
         return;
       }
@@ -2362,10 +2374,7 @@ var audit12 = {
       inefficientImageFormatCount++;
       offendingElementIds.add(node.id);
     };
-    walkImageLikeSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
-    walkBackgroundImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
-    walkAtomicImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
-    walkAtomicBackgroundImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
+    walkAllImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
     if (!hasAnyImage) {
       return { status: "skipped", reason: __30("No images", "elementor") };
     }
@@ -2415,10 +2424,7 @@ var audit13 = {
       const currentMax = widgetMaxKb.get(node.id) ?? 0;
       widgetMaxKb.set(node.id, Math.max(currentMax, kb));
     };
-    walkImageLikeSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
-    walkBackgroundImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
-    walkAtomicImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
-    walkAtomicBackgroundImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
+    walkAllImageSources(ctx.elements.tree, ({ node, media }) => evaluateSource(node, media));
     if (!hasAnyImage) {
       return { status: "skipped", reason: __31("No images", "elementor") };
     }
@@ -2966,50 +2972,6 @@ var audit24 = {
   }
 };
 
-// src/audits/too-many-widgets.ts
-import { getWidgetsCache as getWidgetsCache3 } from "@elementor/editor-elements";
-import { __ as __43 } from "@wordpress/i18n";
-var WIDGET_COUNT_THRESHOLD = 100;
-function isCountableWidget(node, widgetsCache) {
-  if (node.elType === "widget") {
-    return true;
-  }
-  return !!widgetsCache?.[node.elType]?.meta?.is_compound;
-}
-var audit25 = {
-  id: "audits/too-many-widgets",
-  title: __43("Too many widgets", "elementor"),
-  description: __43("Excessive DOM size caused by too many widgets degrades rendering performance.", "elementor"),
-  fixHint: __43("Reduce the number of widgets on the page by removing or combining elements.", "elementor"),
-  categories: ["best-practices", "performance"],
-  severity: "warning",
-  weight: 5,
-  evaluate: (ctx) => {
-    if (ctx.elements.tree.length === 0) {
-      return { status: "skipped", reason: __43("No elements", "elementor") };
-    }
-    const widgetsCache = getWidgetsCache3();
-    let widgetCount = 0;
-    walkElements(ctx.elements.tree, (node) => {
-      if (isCountableWidget(node, widgetsCache)) {
-        widgetCount++;
-      }
-    });
-    if (widgetCount <= WIDGET_COUNT_THRESHOLD) {
-      return { status: "pass" };
-    }
-    return {
-      status: "fail",
-      violations: [
-        {
-          auditId: audit25.id,
-          label: __43("Page has too many widgets.", "elementor")
-        }
-      ]
-    };
-  }
-};
-
 // src/register-audits.ts
 var AUDITS = [
   audit17,
@@ -3017,7 +2979,6 @@ var AUDITS = [
   audit16,
   audit10,
   audit7,
-  audit25,
   audit23,
   audit5,
   audit14,
@@ -3039,8 +3000,8 @@ var AUDITS = [
   audit8
 ];
 function registerAllAudits() {
-  for (const audit26 of AUDITS) {
-    registerAudit(audit26);
+  for (const audit25 of AUDITS) {
+    registerAudit(audit25);
   }
 }