-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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 never-supported Window features #6942
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went through to verify that these results are accurate. Looks like two features were actually supported in older versions of Firefox.
It has never been implemented, see mdn/browser-compat-data#6942.
This isn't exposed to web contents. There isn't anything that it would be useful to redirect to, so also remove existing redirects to the removed page. Needed for mdn/browser-compat-data#6942.
@@ -1163,55 +1163,6 @@ | |||
} | |||
} | |||
}, | |||
"clipboardchange_event": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can find no trace of this event in Chromium, which is where it might have been given the maintainers of https://w3c.github.io/clipboard-apis/#clipboard-event-clipboardchange.
"clipboardchange_event": { | ||
"__compat": { | ||
"description": "<code>clipboardchange</code> event", | ||
"mdn_url": "https://developer.mozilla.org/docs/Web/API/Window/clipboardchange_event", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've sent mdn/content#1301 to remove this page.
@@ -2640,102 +2591,6 @@ | |||
} | |||
} | |||
}, | |||
"getAttention": { | |||
"__compat": { | |||
"mdn_url": "https://developer.mozilla.org/docs/Web/API/Window/getAttention", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've sent mdn/content#1302 to remove this page.
}, | ||
"getAttentionWithCycleCount": { | ||
"__compat": { | ||
"mdn_url": "https://developer.mozilla.org/docs/Web/API/Window/getAttentionWithCycleCount", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This page doesn't exist, good.
api/Window.json
Outdated
@@ -10069,55 +9924,6 @@ | |||
} | |||
} | |||
}, | |||
"vrdisplayblur_event": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like this was in Edge: https://docs.microsoft.com/en-us/previous-versions/mt801971(v%3Dvs.85)
I'll send a PR to copy the data from the onvrdisplayblur" entry.
This isn't exposed to web contents. There isn't anything that it would be useful to redirect to, so also remove existing redirects to the removed page. Needed for mdn/browser-compat-data#6942.
It has never been implemented, see mdn/browser-compat-data#6942.
This isn't exposed to web contents. There isn't anything that it would be useful to redirect to, so also remove existing redirects to the removed page. Needed for mdn/browser-compat-data#6942.
It has never been implemented, see mdn/browser-compat-data#6942.
A checklist to help your pull request get merged faster: