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

css.properties.filter - SVG support data is incorrect #18917

Closed
estelle opened this issue Feb 15, 2023 · 1 comment · Fixed by #21569
Closed

css.properties.filter - SVG support data is incorrect #18917

estelle opened this issue Feb 15, 2023 · 1 comment · Fixed by #21569
Labels
data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS

Comments

@estelle
Copy link
Member

estelle commented Feb 15, 2023

What type of issue is this?

Incorrect support data (ex. Chrome says "86" but support was added in "40")

What information was incorrect, unhelpful, or incomplete?

There is a BCD support row for the filter property "On SVG elements".

filter works on <SVG> elements and the<path> element in Edge and other browsers. See https://codepen.io/estelle/pen/rNZNqaK. I am not sure if this line is incorrect. Or, if support is correct, to what this line is referring.

What browsers does this problem apply to, if applicable?

No response

What did you expect to see?

A more information table row header or more accurate support information

Did you test this? If so, how?

Created a codepen and looked at it in other browsers. https://codepen.io/estelle/pen/rNZNqaK

Can you link to any release notes, bugs, pull requests, or MDN pages related to this?

No response

Do you have anything more you want to share?

No response

MDN URL

https://developer.mozilla.org/en-US/docs/Web/CSS/filter

MDN metadata

MDN page report details
  • Query: css.properties.filter
  • Report started: 2023-02-15T00:46:24.911Z
@queengooborg queengooborg added the data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label Feb 15, 2023
@queengooborg queengooborg changed the title css.properties.filter - unsure what 2nd line of BCD refers to. css.properties.filter - SVG support data is incorrect Dec 10, 2023
queengooborg added a commit to queengooborg/browser-compat-data that referenced this issue Dec 10, 2023
This PR updates and corrects version values for Chrome and Safari for the `svg` member of the `filter` CSS property. This fixes mdn#18917, which contains the supporting evidence for this change.
Elchi3 pushed a commit that referenced this issue Dec 12, 2023
This PR updates and corrects version values for Chrome and Safari for the `svg` member of the `filter` CSS property. This fixes #18917, which contains the supporting evidence for this change.
@foolip
Copy link
Collaborator

foolip commented Jul 19, 2024

I found my way here via #21569 when trying to determine exact versions and remove the ranged versions.

Chrome didn't support the CSS filter property on SVG elements, tracked by this bug, fixed in Chrome 89:
https://issues.chromium.org/u/1/issues/40134524

Testing https://codepen.io/estelle/pen/rNZNqaK in Chrome 88 and 89 confirms this. I also tested https://software.hixie.ch/utilities/js/live-dom-viewer/?saved=12923 and got the same results, it starts working in Chrome 89.

The WebKit bug remains open: https://bugs.webkit.org/show_bug.cgi?id=246106

I'll send a PR to fix this data.

foolip added a commit to foolip/browser-compat-data that referenced this issue Jul 19, 2024
This was most recently researched and updated here:
mdn#18917
mdn#21569

These two tests were run in Chrome 88, 89, Edge 18, and Safari 17.5:
https://codepen.io/estelle/pen/rNZNqaK
https://software.hixie.ch/utilities/js/live-dom-viewer/?saved=12923

The filter works in Chrome 89, but not in Chrome 88, Edge 18, or Safari
17.5.

Chrome 89 is confirmed by these additional sources:
https://stackoverflow.com/questions/32567156/why-dont-css-filters-work-on-svg-elements-in-chrome
https://crbug.com/40134524#comment79

Lack of support in Safari is confirmed by these sources:
https://stackoverflow.com/questions/76919922/css-filters-on-svg-elements-not-applied-in-safari
https://bugs.webkit.org/show_bug.cgi?id=246106

The WebKit bug is linked.
foolip added a commit to foolip/browser-compat-data that referenced this issue Jul 19, 2024
This was most recently researched and updated here:
mdn#18917
mdn#21569

These two tests were run in Chrome 88, 89, Edge 18, and Safari 17.5:
https://codepen.io/estelle/pen/rNZNqaK
https://software.hixie.ch/utilities/js/live-dom-viewer/?saved=12923

The filter works in Chrome 89, but not in Chrome 88, Edge 18, or Safari
17.5.

Chrome 89 is confirmed by these additional sources:
https://stackoverflow.com/questions/32567156/why-dont-css-filters-work-on-svg-elements-in-chrome
https://crbug.com/40134524#comment79

Lack of support in Safari is confirmed by these sources:
https://stackoverflow.com/questions/76919922/css-filters-on-svg-elements-not-applied-in-safari
https://bugs.webkit.org/show_bug.cgi?id=246106

The WebKit bug is linked.
Elchi3 pushed a commit that referenced this issue Jul 22, 2024
This was most recently researched and updated here:
#18917
#21569

These two tests were run in Chrome 88, 89, Edge 18, and Safari 17.5:
https://codepen.io/estelle/pen/rNZNqaK
https://software.hixie.ch/utilities/js/live-dom-viewer/?saved=12923

The filter works in Chrome 89, but not in Chrome 88, Edge 18, or Safari
17.5.

Chrome 89 is confirmed by these additional sources:
https://stackoverflow.com/questions/32567156/why-dont-css-filters-work-on-svg-elements-in-chrome
https://crbug.com/40134524#comment79

Lack of support in Safari is confirmed by these sources:
https://stackoverflow.com/questions/76919922/css-filters-on-svg-elements-not-applied-in-safari
https://bugs.webkit.org/show_bug.cgi?id=246106

The WebKit bug is linked.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants