Bug 78535
| Summary: | Remove or deprecate tab group role | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Aaron Leventhal <aaronlevbugs> |
| Component: | Accessibility | Assignee: | Aaron Leventhal <aaronlevbugs> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | Normal | CC: | aaronlevbugs, dmazzoni, jcraig |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Aaron Leventhal
There are two redundant role constants defined - tab group and tab list. Only tab list is actually used in the code. We should remove tab group to help avoid confusion.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
James Craig
WebCore only uses the "tablist" role:
.//Source/WebCore/accessibility/AccessibilityObject.cpp:
1544: { "tablist", TabListRole },
Do you have a reference to a file where you see this other constant defined?
If you're talking about the platform role "AXTabGroup", that predates ARIA. If you're talking about the AXRoleDescription "tab group" (which is not actually spoken by VO b/c VO lands on the Tabs, not the group), that's a platform-specific localized user string, and matches conventions for the OS X platform. That string is actually pulled from AppKit, so it's not even in the WebKit source.
I believe this bug should be closed as RESOLVED > WORKSFORME. Sending to Aaron to close or will close later if no response received.
James Craig
Closing