Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create roles-minimum.tentative.html #44934

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

scottaohara
Copy link
Contributor

add tests for minimum role in html aam
related to w3c/html-aam#454

add tests for minimum role in html aam
w3c/html-aam#454
html-aam/roles-minimum.html Outdated Show resolved Hide resolved
</head>
<body>

<p>Tests minimum computedrole mappings defined in <a href="https://w3c.github.io/html-aam/#minimum-role">HTML-AAM</a>, where the returned computed role for generic elements is expected to change based on the generic elements having attributes that would require the element no longer be generic, and potentially ignored by browsers. Most test names correspond to a unique ID defined in the spec.<p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this isn't landed yet (dead spec link), I'd expect to see a link to the issue that cross-references the proposed spec diff, implementation trackers, etc.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue link could be removed at the time that everything is merged and the test file loses its tentative status.

change file name to include .tentative
@cookiecrook
Copy link
Contributor

Kicking the WPT CI bots by closing/reopening.

@cookiecrook cookiecrook reopened this Apr 18, 2024
@scottaohara scottaohara changed the title Create roles-minimum.html Create roles-minimum.tentitive.html May 8, 2024
@cookiecrook cookiecrook changed the title Create roles-minimum.tentitive.html Create roles-minimum.tentative.html Jun 14, 2024
html-aam/roles-minimum.tentative.html Outdated Show resolved Hide resolved
@cookiecrook
Copy link
Contributor

Will merge once the CI tests complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants