[webkit-changes] [WebKit/WebKit] c69c97: AX: details/summary not included in VoiceOver Form...
Tyler Wilcock
noreply at github.com
Sun Oct 22 23:14:16 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: c69c97c046fd5b44688eb842e644ae3cb36c5bd2
https://github.com/WebKit/WebKit/commit/c69c97c046fd5b44688eb842e644ae3cb36c5bd2
Author: Tyler Wilcock <tyler_w at apple.com>
Date: 2023-10-22 (Sun, 22 Oct 2023)
Changed paths:
A LayoutTests/accessibility/ios-simulator/activate-summary-expected.txt
A LayoutTests/accessibility/ios-simulator/activate-summary.html
A LayoutTests/accessibility/mac/search-predicate-summary-expected.txt
A LayoutTests/accessibility/mac/search-predicate-summary.html
M Source/WebCore/accessibility/AccessibilityNodeObject.cpp
M Source/WebCore/accessibility/AccessibilityNodeObject.h
M Source/WebCore/accessibility/AccessibilityObject.cpp
M Source/WebCore/accessibility/AccessibilityObject.h
M Source/WebCore/accessibility/AccessibilityRenderObject.cpp
M Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm
Log Message:
-----------
AX: details/summary not included in VoiceOver Form Controls menu or list
https://bugs.webkit.org/show_bug.cgi?id=263499
rdar://problem/117308226
Reviewed by Chris Fleizach.
This is fixed on iOS by exposing summary text as a button (as pressing it toggles the containing details open state).
On macOS, AXControlSearchKey is modified to include summary elements (consistent with the fact that we expose them as
buttons prior to this patch). We also modify AccessibilityObject::supportsPressAction to help indicate their interacive
state.
This patch also fixes a bug on iOS where a details element with a user-agent default summary did not expand or collapse
when activated via VoiceOver.
* LayoutTests/accessibility/ios-simulator/activate-summary-expected.txt: Added.
* LayoutTests/accessibility/ios-simulator/activate-summary.html: Added.
* LayoutTests/accessibility/mac/search-predicate-summary-expected.txt: Added.
* LayoutTests/accessibility/mac/search-predicate-summary.html: Added.
* Source/WebCore/accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::toggleDetailsAncestor):
* Source/WebCore/accessibility/AccessibilityNodeObject.h:
* Source/WebCore/accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::supportsPressAction const):
(WebCore::Accessibility::isAccessibilityObjectSearchMatchAtIndex):
* Source/WebCore/accessibility/AccessibilityObject.h:
(WebCore::AccessibilityObject::toggleDetailsAncestor):
* Source/WebCore/accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::computeAccessibilityIsIgnored const):
* Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:
(-[WebAccessibilityObjectWrapper _accessibilityTraitsFromAncestors]):
(-[WebAccessibilityObjectWrapper accessibilityTraits]):
(-[WebAccessibilityObjectWrapper _accessibilityActivate]):
Canonical link: https://commits.webkit.org/269643@main
More information about the webkit-changes
mailing list