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

[html-aam PR 529] Addition: Name from heading #2215

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

Conversation

pkra
Copy link
Member

@pkra pkra commented May 23, 2024

Moved from w3c/html-aam#529

Adds name from heading computation steps for <article> and <dialog>

scottaohara and others added 5 commits February 6, 2024 11:28
closes #457

Related to the following:
- #1860
- w3c/accname#229 (this needs to be merged so the new links to 'accName: name from heading' will work
Co-authored-by: James Craig <[email protected]>
Co-authored-by: James Craig <[email protected]>
per @cookiecrook review, changing a couple more instances where 'accessible name' should be used instead of 'subtree'
@pkra pkra changed the base branch from monorepo_history--html-aam to main June 12, 2024 19:29
@pkra pkra changed the title [Monorepo] [html-aam PR 529] Addition: Name from heading [html-aam PR 529] Addition: Name from heading Jun 12, 2024
@rahimabdi
Copy link
Contributor

Re-assigning reviewers @cookiecrook @spectranaut for this migrated html-aam PR.

Copy link
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

The text looks good to me. @cookiecrook @scottaohara I wonder if these changes should be moved to the main PR: #1860 -- since it is related to the implementation and should land at the same time?

That is the goal of the mono repo, less PRs when there are dependent changes across specs.

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

Successfully merging this pull request may close these issues.

4 participants