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

a11y: test empty string fallback for CSS content property #44971

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

annevk
Copy link
Member

@annevk annevk commented Mar 7, 2024

No description provided.

@annevk annevk enabled auto-merge (rebase) March 9, 2024 07:40
webkit-commit-queue pushed a commit to annevk/WebKit that referenced this pull request Mar 9, 2024
…y string

https://bugs.webkit.org/show_bug.cgi?id=270377
rdar://123919677

Reviewed by Antoine Quint.

This issue came to light as part of the <input type=checkbox switch>
demo, but is more universally applicable as shown in the tests.

While here, also cleanup some of the code as the render objects
involved cannot become null.

New web-platform-tests tests are exported:
web-platform-tests/wpt#44971

* LayoutTests/accessibility/mac/alt-for-css-content-expected.txt:
* LayoutTests/accessibility/mac/alt-for-css-content.html:
* LayoutTests/imported/w3c/web-platform-tests/accname/name/comp_name_from_content-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/accname/name/comp_name_from_content.html:
* Source/WebCore/accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::textUnderElement const):

Canonical link: https://commits.webkit.org/275872@main
@annevk annevk merged commit d9a5cc2 into master Mar 9, 2024
18 checks passed
@annevk annevk deleted the annevk/css-content-empty-fallback branch March 9, 2024 07:44
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.

None yet

5 participants