Skip to content

Commit

Permalink
Fix animateColor test to work across browsers (web-platform-tests#4…
Browse files Browse the repository at this point in the history
…5015)

Hi Team,

This PR is to update `animateColor` SVG animation tests from SVG1.1
suite to align with all browsers.

Blink removed it with following in 2014 - https://src.chromium.org/viewvc/blink?view=revision&revision=165071

Firefox / Gecko - never supported it (https://bugzilla.mozilla.org/show_bug.cgi?id=436296)
^ Since all use cases can be done via `animate`

Safari / WebKit - trying to remove - WebKit/WebKit#25671

Just raising this PR so we fix the tests on upstream as well, so on
next import, we don't have to fix them and all future engines importing
also align with other browsers.

Thanks!
  • Loading branch information
Ahmad-S792 authored Mar 11, 2024
1 parent 453e2ca commit f363ae7
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion svg/import/animate-elem-30-t-manual.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
18 changes: 9 additions & 9 deletions svg/import/animate-elem-85-t-manual.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
10 changes: 5 additions & 5 deletions svg/import/animate-pservers-grad-01-b-manual.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit f363ae7

Please sign in to comment.