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

Fix info for svg <style> element #37288

Merged
merged 3 commits into from
Dec 22, 2024
Merged

Conversation

skyclouds2001
Copy link
Contributor

@skyclouds2001 skyclouds2001 commented Dec 20, 2024

Description

media attribute ( for <style> element) is not animatable
update type for <style> attributes

Motivation

Additional details

https://svgwg.org/svg2-draft/styling.html#StyleElement

Related issues and pull requests

@github-actions github-actions bot added Content:SVG SVG docs size/s [PR only] 6-50 LoC changed labels Dec 20, 2024
Copy link
Contributor

github-actions bot commented Dec 20, 2024

Preview URLs

(comment last updated: 2024-12-20 08:34:25)

@skyclouds2001 skyclouds2001 marked this pull request as ready for review December 20, 2024 02:59
@skyclouds2001 skyclouds2001 requested a review from a team as a code owner December 20, 2024 02:59
@skyclouds2001 skyclouds2001 requested review from hamishwillee and removed request for a team December 20, 2024 02:59
@@ -42,10 +42,10 @@ svg {

- {{SVGAttr("type")}}
- : This attribute defines type of the style sheet language to use as a media type string.
_Value type_: [**\<string>**](/en-US/docs/Web/CSS/string); _Default value_: `text/css`; _Animatable_: **no**
_Value type_: [**`<content-type>`**](https://www.ietf.org/rfc/rfc2046.txt); _Default value_: `text/css`; _Animatable_: **no**
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure about this change as we prefer not to link to specs. Could we instead link to https://developer.mozilla.org/en-US/docs/Glossary/MIME_type and call it a<media-type> (the spec does)?

The rest looks good.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P.S, mostly I think the linked doc is much more accessible than the spec.

Copy link
Collaborator

@hamishwillee hamishwillee Dec 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P.S. Just FYI @skyclouds2001 As I think I noted elswhere, I am here Monday, maybe Tuesday but then will be away a lot until Jan 20. So if something gets auto-assigned to me, you will either have to find someone else to look at it or wait. I don't want to hold you up!

Copy link
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much @skyclouds2001 !

@hamishwillee hamishwillee merged commit bc780f1 into mdn:main Dec 22, 2024
8 checks passed
@skyclouds2001 skyclouds2001 deleted the svg-style branch December 23, 2024 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:SVG SVG docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants