-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
add: instance methods SVGAnimationElement #37180
add: instance methods SVGAnimationElement #37180
Conversation
Preview URLs
(comment last updated: 2025-01-12 04:14:20) |
files/en-us/web/api/svganimationelement/getcurrenttime/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/api/svganimationelement/getsimpleduration/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/api/svganimationelement/getsimpleduration/index.md
Outdated
Show resolved
Hide resolved
Thanks for the review! I will start fixing these right away, as a question, should I also make these changes to the SVGAnimationElement Web API page as they are consistent with the description there. |
Oh, I didn't know this was the same as the interface page. Yes, if you can find a better way to express this then it would be good to update that page as well. However if the explanation is long, it's better not to include the full explanation in the interface page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 thank you @yashrajbharti , this looks great :).
part of Missing baseline pages project