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

Remove irrelevant SVGZoomAndPan API #12518

Merged
merged 1 commit into from
Sep 21, 2021

Conversation

queengooborg
Copy link
Collaborator

This PR removes the irrelevant SVGZoomAndPan API as per the corresponding data guidelines. The lack of current support has been confirmed by the mdn-bcd-collector project (v3.3.0), even if the current BCD suggests support.

This PR removes the irrelevant `SVGZoomAndPan` API as per the corresponding [data guidelines](https://github.com/mdn/browser-compat-data/blob/main/docs/data-guidelines.md#removal-of-irrelevant-features). The lack of current support has been confirmed by the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v3.3.0), even if the current BCD suggests support.
@queengooborg queengooborg added needs-release-note 📰 needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation labels Sep 21, 2021
@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Sep 21, 2021
@queengooborg queengooborg removed the needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation label Sep 21, 2021
@foolip
Copy link
Collaborator

foolip commented Sep 21, 2021

I've confirmed with https://mdn-bcd-collector.appspot.com/tests/api/SVGZoomAndPan that SVGZoomAndPan isn't its own exposed interface in Edge 18, Chrome 80 or Safari 12.1.

It was however in Firefox 70, so it's not old enough to remove under the irrelevant features guideline. I think it should be removed anyway since it's kind of useless without also having data for the zoomAndPan properties that the constants on this interface were used for, but that would be a different kind of reason.

@ddbeck do you think we can remove this half-present and almost-old-enough-for-removal feature?

@foolip
Copy link
Collaborator

foolip commented Sep 21, 2021

@queengooborg maybe you can make "zoomAndPan" attribute removal into this as well? Those are present, but the properties by the same name aren't.

@ddbeck
Copy link
Collaborator

ddbeck commented Sep 21, 2021

@ddbeck do you think we can remove this half-present and almost-old-enough-for-removal feature?

We're what, a month off and the API is useless? Go ahead. 👍

@foolip
Copy link
Collaborator

foolip commented Sep 21, 2021

Alright, breaking the rules with abandon!

@foolip foolip merged commit a2ad02f into mdn:main Sep 21, 2021
@queengooborg queengooborg deleted the api/SVGZoomAndPan/removal branch September 21, 2021 16:36
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Sep 22, 2021
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Sep 22, 2021
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Sep 22, 2021
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Sep 22, 2021
ddbeck added a commit that referenced this pull request Sep 23, 2021
* Bump version to v4.0.5

* Bump known issues

* Add release note for #12417

* Add release note for #12454

* Add release note for #12417

* Add release note for #12346

* Add release note for #12455

* Add release note for #12518

* Add release note for #11630

* Add release note for #12300

* Add release note for #12552

* Add release stats

* Add release date

* Fix a typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants