Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

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

Browse source 
Fixes https://github.com/microsoft/playwright/issues/31254
This commit is contained in:
Yury Semikhatsky 2024-06-17 16:32:22 -07:00 committed by GitHub
parent f3f708e01a
commit 5443b66636
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/playwright-core/src/server/injected/selectorGenerator.ts
View file

@ -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 });
}
}