-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clean up language re <title>/<desc> and tooltips #102
Comments
(and clean-up re SVG accessibility issues for empty title/desc and multilingual) |
Removed issue from the spec relating to taking text from SVG Tiny 1.2 in commit 41fa130 |
Also need to clean up re which elements can have a title/desc child. Currently the prose says "container" and "graphics" elements can have title & desc, but I think other elements allow descriptive elements in their content models, while |
This is editorial, but needs to be rewritten in sync with the accessibility specs. |
Cleaning up which elements can have title/desc extends to the FXTF specs, see w3c/fxtf-drafts#9 |
Besides container elements (excluding |
My instinct is to allow them generously, except for where there is a direct logical conflict (like
The phrase "blue and white polka dot pattern" would be available as an extra description on the shape because of the The alternative approach would be to forbid |
@AmeliaBR thanks for clarification. I also think the first approach makes more sense now, though if further unification of SVG and HTML is going to happen, I would opt for limiting the places where |
@jarek-foksa Switching to attributes would have the downside of removing the option to create multilingual titles. And besides, we've only finally got the browsers to start treating As @Tavmjong noted on the FXTF thread, we resolved today to "title and desc can be a child of any svg namespaced element except switch". After the fact, I thought of a few additional exceptions, however:
Lots of other elements are unlikely to particularly benefit from descriptive content, but they wouldn't break so there's no reason to forbid it. |
- Move the blue boxes down to be associated with the sections specific to title and desc, respectively (after general rules that apply to both) - Re-arrange a number of sentences/paragraphs to group together normative text for authors vs user agents - Clarify rules & add example for matching multi-lingual descriptive text - SHOULD > MUST for user agent exposing text content to accessibility APIs - Authors SHOULD NOT, authoring tools MUST NOT include empty title/desc (this causes a mess on the accessibility side) - Advice for which elements should have title/desc - Purpose of title/desc on non-rendered content - Clean up the examples of purpose of title & desc - Mention aria-labelledby when warning against redundant title content - User agents SHOULD make titles available to all users on interaction (tooltips still being only one possible way of doing so) - Clarify how this differs in practice for title on the root element - Clean up paragraph on aria-describedby - remove generic example (redundant given the earlier example) - remove inaccurate statements about alternative plain-text presentations - add warning that other namespace markup within title/desc has no affect on accessible alternative text (clears spec issue 23) Addresses a lot of #102, fixes #101
Leaving this issue open until we clean up the content model of all other elements to correctly include or exclude title & desc. |
For the content model, it looks like there's no easy way to say "any possible child element except other title/desc" for the title/desc, so we'll just leave that as is. It doesn't really change anything, since we're already clear that markup inside title & desc is ignored. The rest of the content model (for other elements) has been cleared up: title & desc are allowed anywhere except script, style, & as a direct child of switch. |
https://svgwg.org/svg2-draft/struct.html#issue20
I don't think there's anything important in Tiny 1.2 that isn't in SVG 2. Particularly, with the resolution of issue #100.
I'll open the issue in case anyone else has differing views and we can review in London.
The text was updated successfully, but these errors were encountered: