Bug 132958

Summary: AX:Add Visual Focus Support to SVG elements
Product: WebKit Reporter: Rich Schwerdtfeger <richschwer>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: cfleizach, jcraig, krit, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Example none

Description Rich Schwerdtfeger 2014-05-15 08:20:31 PDT
Add Visual Focus Support to SVG elements
Comment 1 Radar WebKit Bug Importer 2014-05-15 08:21:30 PDT
<rdar://problem/16926525>
Comment 2 Rich Schwerdtfeger 2014-05-15 08:26:12 PDT
Visual focus is rendered when I run VoiceOver so focus is advancing. It does not render visual focus when I select the feature to do so in response to tabbing when VoiceOver is running.
Comment 3 Dirk Schulze 2014-05-15 09:20:53 PDT
Created attachment 231523 [details]
Example

Here is a little example that I used. System focus ring and outline work in SVG in general. There is just one bug with text elements that get outline but no system focus ring.
Comment 4 chris fleizach 2014-05-15 11:23:38 PDT
(In reply to comment #3)
> Created an attachment (id=231523) [details]
> Example
> 
> Here is a little example that I used. System focus ring and outline work in SVG in general. There is just one bug with text elements that get outline but no system focus ring.

Dirk, are you going to work on this one?
Comment 5 Dirk Schulze 2014-05-15 11:50:51 PDT
(In reply to comment #4)
> (In reply to comment #3)
> > Created an attachment (id=231523) [details] [details]
> > Example
> > 
> > Here is a little example that I used. System focus ring and outline work in SVG in general. There is just one bug with text elements that get outline but no system focus ring.
> 
> Dirk, are you going to work on this one?

No, not at the moment.
Comment 6 Dirk Schulze 2014-05-15 11:53:43 PDT
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > Created an attachment (id=231523) [details] [details] [details]
> > > Example
> > > 
> > > Here is a little example that I used. System focus ring and outline work in SVG in general. There is just one bug with text elements that get outline but no system focus ring.
> > 
> > Dirk, are you going to work on this one?
> 
> No, not at the moment.

I can at least say that we indeed never called SVGInlineTextBox::paint with paint phase Outline. Even if that does not affect system focus, I could imagine that we don't call it for the same reason (which I don't know atm).
Comment 7 James Craig 2014-06-20 10:41:14 PDT
Another test case.  It could probably be updated to use drawFocusRing.
http://www.html5accessibility.com/tests/canvas.html