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.

Sign up for GitHub

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

Jump to bottom

Attempt to split attribute by type #485

Merged
merged 4 commits into from
Jun 3, 2024
Merged

Attempt to split attribute by type #485

merged 4 commits into from
Jun 3, 2024

Conversation

martinthomson
Copy link
Contributor

@martinthomson martinthomson commented Apr 1, 2024 •
edited by pr-preview bot

HTML vs. IDL

Preview | Diff

@martinthomson martinthomson linked an issue Apr 1, 2024 that may be closed by this pull request
Use of "attribute" confusing/ambiguous #457
Closed
LeaVerou
LeaVerou reviewed Apr 2, 2024
View reviewed changes
index.bs Outdated Show resolved Hide resolved
@martinthomson @LeaVerou
Accessors
2ff7be2 
Co-authored-by: Lea Verou <lea@verou.me>
martinthomson
martinthomson commented Apr 2, 2024
View reviewed changes
index.bs Outdated Show resolved Hide resolved
@annevk
Copy link
Member

annevk commented Apr 3, 2024

The HTML Standard uses "content attributes" and "IDL attributes". Not a big fan of that terminology, but at least you don't rule out SVG and MatML immediately.

@LeaVerou
Copy link
Member

LeaVerou commented Apr 3, 2024

The HTML Standard uses "content attributes" and "IDL attributes". Not a big fan of that terminology, but at least you don't rule out SVG and MatML immediately.

Ooh good point, we definitely don't want to exclude those.
Some other ideas:

  • markup attributes
  • element attributes

@torgo torgo added this to the 2024-06-03-week milestone Jun 2, 2024
@martinthomson
Copy link
Contributor Author

@LeaVerou is there a problem with "HTML attribute" in the context that it appears? @rhiaro suggests that this is an improvement, even if it isn't perfect.

martinthomson
martinthomson commented Jun 3, 2024
View reviewed changes
index.bs Outdated Show resolved Hide resolved
@plinss plinss merged commit c3eaec9 into w3ctag:main Jun 3, 2024
@martinthomson martinthomson mentioned this pull request Jun 3, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LeaVerou LeaVerou LeaVerou left review comments

@rhiaro rhiaro rhiaro approved these changes

@plinss plinss plinss approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2024-06-03-week
Development

Successfully merging this pull request may close these issues.

Use of "attribute" confusing/ambiguous
6 participants
@martinthomson @annevk @LeaVerou @rhiaro @plinss @torgo