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

normalize “an empty string” to “the empty string” #544

Merged
merged 7 commits into from
Apr 18, 2024

Conversation

adampage
Copy link
Member

@adampage adampage commented Apr 18, 2024

Closes #514.

This normalizes all instances of “an empty string” to “the empty string”.

I’ve opened an equivalent ARIA PR for the ARIA spec in w3c/aria#2163.

I checked accname and confirmed that it already uses “the empty string” consistently throughout.


Preview | Diff

test.html Outdated Show resolved Hide resolved
test.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
adampage and others added 2 commits April 18, 2024 08:17
Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
test.html Outdated Show resolved Hide resolved
adampage and others added 2 commits April 18, 2024 08:17
Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
@scottaohara
Copy link
Member

thanks for this (and immediately pulling in my nits for commas that are beyond your PR :) )

pkra pushed a commit to w3c/aria that referenced this pull request Apr 18, 2024
Related to w3c/html-aam#514

This normalizes all instances of “_**an** empty string_” to “_**the** empty string_”.

I’ve opened an equivalent HTML-AAM PR in w3c/html-aam#544.

I checked accname and confirmed that it already uses “_the empty string_” consistently throughout.
@scottaohara scottaohara merged commit ff5bdf3 into w3c:gh-pages Apr 18, 2024
2 checks passed
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.

Should it say "the empty string" instead of "an empty string"
2 participants