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: Fix the element <animate> #306

Merged
merged 4 commits into from Sep 27, 2023
Merged

Conversation

ilankri
Copy link
Member

@ilankri ilankri commented Apr 12, 2022

Fix #305.

@ilankri ilankri self-assigned this Apr 12, 2022
@ilankri ilankri marked this pull request as ready for review April 12, 2022 15:16
@ilankri ilankri marked this pull request as draft April 12, 2022 15:17
@ilankri ilankri marked this pull request as ready for review April 12, 2022 16:05
@ilankri ilankri marked this pull request as draft April 12, 2022 16:47
@ilankri ilankri marked this pull request as ready for review April 12, 2022 17:33
@rand00
Copy link

rand00 commented Jun 20, 2022

Didn't see this PR before making my own fixes. Though as I now skimmed the code of this PR, I didn't see a fix for a_animation_values, a_keyTimes and a_keySplines - which all need to be semicolon separated instead of comma-separated. My working version is here: https://github.com/rand00/tyxml

@Drup
Copy link
Member

Drup commented Jun 21, 2022

oups, looks like I missed these bug reports and PRs. @rand00 Can you make a PR with the package of everything ?

@rand00
Copy link

rand00 commented Jun 21, 2022

Yes - I'll merge the PRs together in a moment

@Drup Drup merged commit c05cb5c into ocsigen:master Sep 27, 2023
Drup added a commit to Drup/opam-repository that referenced this pull request Sep 29, 2023
CHANGES:

* Update for OCaml 5.0 and drop support for OCaml 4.2.0
  (ocsigen/tyxml#312 by @rr0gi)

* Add additional variants to `linktype` for the `rel` attribute
  (Leon @LogicalOverflow Vack)

* Expand options for `autocomplete` attribute on `<input>` elements
  (ocsigen/tyxml#302 by Aron @aronerben Erben)

* Fix the SVG element `<animate>` (by the way, deprecate `animation` et
  al. in favor of `animate` et al.)
  (ocsigen/tyxml#306 by Idir @ilankri Lankri)

* Add support for `dialog` element and `onclose` attribute
  (ocsigen/tyxml#301 by Julien Sagot)
* Add an escape hatch for emitting attributes with non-standard names
  in jsx or ppx code (a leading `_` character on attribute name)
  (ocsigen/tyxml#295 Chas @cemerick Emerick)
* Add support for `type` attribute on `<script>` elements
  (ocsigen/tyxml#293 by Ulrik @ulrikstrid Strid and Chas @cemerick Emerick)

* Add svg `fill-rule` attribute
  (ocsigen/tyxml#294 by Eric @dedbox Griffis)
@ilankri ilankri deleted the svg-fix-animate branch December 16, 2023 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to add <animate /> in <circle> ?
3 participants