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

fix(retarget): do not unconditionally retarget to enclosing label

Browse source
This commit is contained in:
Pavel Feldman 2025-01-06 17:10:53 -08:00
parent 4e8c83055f
commit 3063538470
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
packages/playwright-core/src/server/injected/injectedScript.ts
View file

@ -531,10 +531,10 @@ export class InjectedScript {
if (!element.matches('a, input, textarea, button, select, [role=link], [role=button], [role=checkbox], [role=radio]') &&
!(element as any).isContentEditable) {
// Go up to the label that might be connected to the input/textarea.
element = element.closest('label') || element;
const enclosingLabel: HTMLLabelElement | null = element.closest('label');
if (enclosingLabel && enclosingLabel.control)
element = enclosingLabel.control;
}
if (element.nodeName === 'LABEL')
element = (element as HTMLLabelElement).control || element;
}
return element;
}

2
tests/page/retarget.spec.ts
View file

@ -109,6 +109,7 @@ it('enabled/disabled retargeting', async ({ page, asset }) => {
{ dom: domInButton(`<input id=target>`), enabled: true, locator: 'input' },
{ dom: domInLink(`<input id=target>`), enabled: true, locator: 'input' },
{ dom: domInButton(`<input id=target>`, { disabled: true }), enabled: true, locator: 'input' },
{ dom: domInLabel(`<li role=menuitem id=target aria-disabled=false></li>`), enabled: true, locator: 'li' },
{ dom: domInLabel(`<input id=target disabled>`), enabled: false, locator: 'label' },
{ dom: domLabelFor(`<input id=target disabled>`), enabled: false, locator: 'label' },
@ -116,6 +117,7 @@ it('enabled/disabled retargeting', async ({ page, asset }) => {
{ dom: domInButton(`<input id=target disabled>`), enabled: false, locator: 'input' },
{ dom: domInLink(`<input id=target disabled>`), enabled: false, locator: 'input' },
{ dom: domInButton(`<input id=target disabled>`, { disabled: true }), enabled: false, locator: 'input' },
{ dom: domInLabel(`<li role=menuitem id=target aria-disabled=true></li>`), enabled: false, locator: 'li' },
];
for (const { dom, enabled, locator } of cases) {
await it.step(`"${locator}" in "${dom}" should be enabled=${enabled}`, async () => {