Skip to content

Remove irrelevant Notation API#10225

Merged
Elchi3 merged 2 commits intomdn:mainfrom
queengooborg:api/Notation/edge
May 6, 2021
Merged

Remove irrelevant Notation API#10225
Elchi3 merged 2 commits intomdn:mainfrom
queengooborg:api/Notation/edge

Conversation

@queengooborg
Copy link
Contributor

@queengooborg queengooborg commented May 3, 2021

This PR removes the Notation API data, as per the removal of irrelevant features guideline. Since this was removed back in Safari 9 and no other browsers have shown signs of support, this API qualifies as an irrelevant feature.

This PR adds real values for Microsoft Edge for the `Notation` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v3.0.1).  Results are manually confirmed for accuracy.

Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/Notation
@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label May 3, 2021
@foolip
Copy link
Contributor

foolip commented May 4, 2021

Do you want to send a PR to delete this file entirely instead, under the irrelevant feature guideline?

@queengooborg queengooborg changed the title Add Edge versions for Notation API Remove irrelevant Notation API May 6, 2021
Copy link
Contributor

@foolip foolip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving but not merging since https://developer.mozilla.org/en-US/docs/Web/API/Notation will also be removing. It's best to merge this at the same time as an MDN PR is merged.

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I opened an MDN PR for this. Thanks for finding and cleaning this! I'm a fan of getting rid of irrelevant entries :)

@Elchi3 Elchi3 merged commit 4928378 into mdn:main May 6, 2021
@queengooborg queengooborg deleted the api/Notation/edge branch May 6, 2021 13:03
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request May 6, 2021
ddbeck added a commit that referenced this pull request May 6, 2021
* Start v3.3.3 release

* Add release note for #9975

* Add release note for #10175

* Add release note for #10225

* Add release note for #10261

* Add release note for #10288

* Add stats
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