764422 - Map svg <g> element to ROLE_GROUP

Status: RESOLVED DUPLICATE of bug 1151648

(Core :: Disability Access APIs, defect)

Description

[David Bolter [:davidb] (NeedInfo me for attention)]

As we add a11y to SVG we'll need to watch for perf hits.

Comment 1

[alexander :surkov (:asurkov)]

Please provide a background on this. Exposing the group role accessible that doesn't group anything (because svg is not accessible in general) doesn't look useful.

Comment 2

[Trevor Saunders (:tbsaunde)]

(In reply to alexander :surkov from comment #1)
> Please provide a background on this. Exposing the group role accessible that
> doesn't group anything (because svg is not accessible in general) doesn't
> look useful.

I'd say this would make sense to do if we want to make svg accessible eventually, but maybe we'd want to hide this behind a pref for now.

Comment 3

[alexander :surkov (:asurkov)]

(In reply to Trevor Saunders (:tbsaunde) from comment #2)
> (In reply to alexander :surkov from comment #1)
> > Please provide a background on this. Exposing the group role accessible that
> > doesn't group anything (because svg is not accessible in general) doesn't
> > look useful.
> 
> I'd say this would make sense to do if we want to make svg accessible
> eventually, but maybe we'd want to hide this behind a pref for now.

If we would need that then we could do it when we did svg accessible, I don't see any utility before this. But even having accessible svg it's not clear whether we need a grouping or not.

Comment 4

[alexander :surkov (:asurkov)]

David, do you work on it (inactivity during half of year)?

Comment 5

[alexander :surkov (:asurkov)]

SVG g element is container element, used to group other SVG elements, can have accessible name and description (http://www.w3.org/TR/SVG/struct.html#Groups)

Comment 6

[David Bolter [:davidb] (NeedInfo me for attention)]

I have a starter patch but nothing worth posting. Higher priorities got in the way. Are you seeing need in the wild?

Note: one of the patches checks mContent->IsSVG(nsGkAtoms::svg) in GetName but I didn't implement the body...

Comment 7

[alexander :surkov (:asurkov)]

Rich had examples but after we implemented accessibility for graphic items then it makes sense to make svg:g accessible too.

Comment 8

[James Teh [:Jamie]]

This was done in bug 1151648.