fix(codegen): trim alt selectors to 80 chars (#31346)

Fixes #31254
microsoft · Jun 17, 2024 · 5443b66 · 5443b66
1 parent f3f708e
commit 5443b66
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/playwright-core/src/server/injected/selectorGenerator.ts b/packages/playwright-core/src/server/injected/selectorGenerator.ts
@@ -528,7 +528,7 @@ function suitableTextAlternatives(text: string) {
     const match = text.match(/^([\d.,]+)[^.,\w]/);
     const leadingNumberLength = match ? match[1].length : 0;
     if (leadingNumberLength) {
-      const alt = text.substring(leadingNumberLength).trimStart();
+      const alt = trimWordBoundary(text.substring(leadingNumberLength).trimStart(), 80);
       result.push({ text: alt, scoreBouns: alt.length <= 30 ? 2 : 1 });
     }
   }
@@ -537,7 +537,7 @@ function suitableTextAlternatives(text: string) {
     const match = text.match(/[^.,\w]([\d.,]+)$/);
     const trailingNumberLength = match ? match[1].length : 0;
     if (trailingNumberLength) {
-      const alt = text.substring(0, text.length - trailingNumberLength).trimEnd();
+      const alt = trimWordBoundary(text.substring(0, text.length - trailingNumberLength).trimEnd(), 80);
       result.push({ text: alt, scoreBouns: alt.length <= 30 ? 2 : 1 });
     }
   }