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

Allow more characters in dataset? #8215

Open
annevk opened this issue Aug 23, 2022 · 1 comment
Open

Allow more characters in dataset? #8215

annevk opened this issue Aug 23, 2022 · 1 comment
Labels
needs tests Moving the issue forward requires someone to write tests normative change

Comments

@annevk
Copy link
Member

annevk commented Aug 23, 2022

We have whatwg/dom#1079 and #7991 in progress to allow more characters in various places, but thus far we did not consider dataset. We probably should for consistency?

According to https://bugs.webkit.org/show_bug.cgi?id=126200 there are also differences here between Blink and WebKit.
@annevk annevk added normative change needs tests Moving the issue forward requires someone to write tests agenda+ To be discussed at a triage meeting labels Aug 23, 2022
@domenic
Copy link
Member

domenic commented Aug 24, 2022

Agreed, this is simple enough. I am happy to update #7991 with the spec changes to check against valid attribute local name instead of the current check against XML Name.

@past past mentioned this issue Sep 1, 2022
Closed
@past past removed the agenda+ To be discussed at a triage meeting label Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs tests Moving the issue forward requires someone to write tests normative change
Milestone
No milestone
Development

No branches or pull requests
3 participants
@past @domenic @annevk