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

SVG XLink attributes are not global attributes #23740

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

Elchi3
Copy link
Member

@Elchi3 Elchi3 commented Jul 10, 2024

The xLink attributes are not global to all SVG elements. They only apply to (many) SVG elements. However, these attributes are deprecated and not part of SVG 2. I think it is fine to remove them. svg.elements.a has some of them and if anyone ever feels strongly, we can accept a PR to add them to the other elements as well, but I think it is not worth it right now.

MDN says the following elements can have these attributes, so this would be a lot of new entries for little value IMO.

<a>
<animate>
<animateMotion>
<animateTransform>
<cursor>
<feImage>
<filter>
<font-face-uri>
<glyphRef>
<image>
<linearGradient>
<mpath>
<pattern>
<radialGradient>
<script>
<set>
<textPath>
<tref>
<use>

@github-actions github-actions bot added the data:svg Compat data for SVG features. https://developer.mozilla.org/docs/Web/SVG label Jul 10, 2024
@queengooborg queengooborg merged commit 7f0ce52 into mdn:main Jul 10, 2024
5 checks passed
@Elchi3 Elchi3 deleted the svg-rm-xlink branch July 10, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:svg Compat data for SVG features. https://developer.mozilla.org/docs/Web/SVG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants