Skip to content
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

Wrong description of "visibility" property #405

Closed
jarek-foksa opened this issue Apr 1, 2018 · 1 comment
Closed

Wrong description of "visibility" property #405

jarek-foksa opened this issue Apr 1, 2018 · 1 comment

Comments

@jarek-foksa
Copy link

jarek-foksa commented Apr 1, 2018

Chapter 3 says that:

The visibility property only directly applies to graphics elements, text content elements (including the ‘a’ element when it is a child of text content element) and ‘use’ elements.

Chapter 5 says that:

The visibility property does not appy directly to container elements, and therefore does not apply directly to the ‘use’ element.

Considering the recent changes to <use> element definition (make it behave like a container with referenced elements cloned into the shadow root), I guess the first sentence is wrong? Also, <a> is not a text content element, so I would avoid using the word "including". Therefore, I would propose to change the sentence from Chapter 3 to:

The visibility property only directly applies to graphics elements, text content elements and "a" element (when it is a child of a text content element).

@AmeliaBR
Copy link
Contributor

AmeliaBR commented Apr 1, 2018

Sounds better.

@boggydigital boggydigital added this to the SVG 2.0 Recommendation milestone Jun 11, 2018
AmeliaBR added a commit that referenced this issue Aug 13, 2018
Fixes #405 update description of when visibility applies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants