[Webkit-unassigned] [Bug 61013] Keep linked images in the accessibility tree.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=61013


Chris Guillory <ctguil at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #94712|commit-queue?               |
               Flag|                            |




--- Comment #5 from Chris Guillory <ctguil at chromium.org>  2011-05-27 15:37:19 PST ---
(From update of attachment 94712)
View in context: https://bugs.webkit.org/attachment.cgi?id=94712&action=review

>> Source/WebCore/ChangeLog:10
>> +
> 
> I may be dense, but this comment doesn't make sense to me. 
> Are you saying... "don't expose the src of the link.... and to do that, you override the image src?"
> a) I don't understand why you don't want to expose the source of the link.
> b) I don't understand why you want to change what the URL is for the image. Right now it should be the the location of the image. That seems correct in my mind, because the src of the link is returned in the link

The comment could use some rewording.
a) The source for links remains unchanged.
b) The change includes images without alt text that are parented to linked in the accessibility tree. However, when a screen reader (JAWS, VoiceOver) focuses the link, the image's src is read. Previously the link's src was read. I think it's incorrect to override the images src to address this behavior. I'll look into an alternative.

>> Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1872
>>                  return true;
> 
> this seems like it alters the informal standard that's pretty clear. if alt is empty, image should be ignored

Firefox and IE include images with alt text in the accessibility tree. JAWS reads them more conveniently as "link graphic" instead of just "link". I'll look more into the general update I'm requesting for the informal standard.

>> LayoutTests/accessibility/tree-contains-linked-images.html:24
>> +     }
> 
> this layout test doesn't seem to test the altered behavior. this <img> was not ignored previously because it had an alt tag.

Oops. I accidentally left that in.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.