Bug 132958 - AX:Add Visual Focus Support to SVG elements
Summary: AX:Add Visual Focus Support to SVG elements
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2014-05-15 08:20 PDT by Rich Schwerdtfeger
Modified: 2014-06-20 10:41 PDT (History)
4 users (show)

See Also:


Attachments
Example (251 bytes, text/html)
2014-05-15 09:20 PDT, Dirk Schulze
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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