Bug 126914 - AX: Do not return an accessible name for an object just because it has tabindex=0
Summary: AX: Do not return an accessible name for an object just because it has tabind...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: chris fleizach
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2014-01-13 10:03 PST by chris fleizach
Modified: 2014-01-22 17:57 PST (History)
9 users (show)

See Also:


Attachments
patch (9.83 KB, patch)
2014-01-13 10:08 PST, chris fleizach
mario: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chris fleizach 2014-01-13 10:03:32 PST
When VoiceOver encounters a <div> that has tabindex=0 on it, it speaks all the text under that div. 

That's because we're concatenating all the text under an element that is focusable. 

There's nothing in ARIA that allows for this and it has caused endless problems for VoiceOver speaking way too much useless information.

I think we should undo this change which isn't supported by any spec.

<rdar://problem/13603051>
Comment 1 chris fleizach 2014-01-13 10:08:34 PST
Created attachment 221059 [details]
patch
Comment 2 Mario Sanchez Prada 2014-01-16 01:55:31 PST
Comment on attachment 221059 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=221059&action=review

The patch looks good to me, although I don't understand why you are getting that undefined expectation for one test. Also, I think we might potentially have to adapt the ATK port after this change, but I'm happy to give it a try now and adapt later so VO is not blocked on theories :)

Setting the r+ anyway, but please take a look to the issue with the FAIL expectation before landing

> LayoutTests/accessibility/focusable-div-expected.txt:17
> -PASS lastChar(axDiv.title) is "B"
> +FAIL lastChar(axDiv.title) should be undefined (of type undefined). Was   (of type string).

This FAIL expectation looks wrong. Not sure why this is happening, though (why is "" undefined?)

> LayoutTests/accessibility/focusable-div-expected.txt:19
> +FAIL lastChar(axDiv2.title) should be undefined (of type undefined). Was   (of type string).

Same here
Comment 3 chris fleizach 2014-01-22 17:57:39 PST
http://trac.webkit.org/changeset/162576