-
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 irrelevant api.Window.beforeunload_event.custom_text_support feature #11753
Remove irrelevant api.Window.beforeunload_event.custom_text_support feature #11753
Conversation
…eature This PR removes the irrelevant `beforeunload_event.custom_text_support` member of the `Window` 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.2.2), even if the current BCD suggests support.
@vinyldarkscratch can you link to the content PR for this? |
There is actually no mention of this subfeature anywhere within the MDN page, so there is no need for a content PR! |
I don't think that's right, or I've badly misunderstood the page and this feature. From the reference page:
|
Wow, that's super embarrassing -- I legitimately have glossed over that paragraph both times I checked the page (and I thought I did a search for "custom" before, but I guess not). Eheh...whoops! A content PR has now been created, and merged just as quickly! |
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.
Thanks, @vinyldarkscratch!
And I hope it wasn't too embarrassing. Honestly, it's a good thing that "reading every word" (or even writing every word) is not a job requirement for being a tech writer, or I'd have a very different career. 😅
* Bump version to v3.3.13 * Add release note for #11717 * Add release note for #11753 * Add release note for #11459 * Add release note for #11720 * Add release note for #11722 * Add release note for #11721 * Add release note for #11763 * Add release note for #10623 * Add release note for #11000 * Add stats and release date
This PR removes the irrelevant
beforeunload_event.custom_text_support
member of theWindow
API as per the corresponding data guidelines. The lack of current support has been confirmed by the mdn-bcd-collector project (v3.2.2), even if the current BCD suggests support.