act-rules / act-rules.github.io

Accessibility conformance testing rules for HTML
https://act-rules.github.io/
Other
136 stars 68 forks source link

[bc659a] Meta element has no refresh delay testcases generated incorrectly #2039

Open tombrunet opened 1 year ago

tombrunet commented 1 year ago

Testcases are being injected into the body element rather than the HTML element. For example:

<head>
    <meta http-equiv="refresh" content="30" />
</head>
<body>
    <p>This page refreshes after 30 seconds.</p>
</body>

is generated as

<!DOCTYPE html>
<html lang="en">
<head>
    <title>Failed Example 1</title>
</head>
<body>
    <head>
        <meta http-equiv="refresh" content="30" />
    </head>
    <body>
        <p>This page refreshes after 30 seconds.</p>
    </body>
</body>
</html>

The resulting html doesn't seem to be the intent of the testcase.

Jym77 commented 1 year ago

IIRC, that should be solved if we include the html element in the test case, then it won't be added incorrectly. We'll need to also keep the lang attribute and the title element, so that the case don't fail other SC 🙈

tombrunet commented 1 year ago

Noticed that the same is happening for bisz58