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

HTML Sanitizer API - delete #35238

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

hamishwillee
Copy link
Collaborator

@hamishwillee hamishwillee commented Jul 30, 2024

This removes the HTML Sanitizer API - specifically Sanitizer and it's methods, and api.Element.setHTML(), and the overview page

The API was added prematurely and briefly and has since been withdrawn everywhere. It is coming back but the spec is significantly modified. If we try and extend this current information it is going to be hard to present a coherent story to users. The API is not quite 2 years out of all browsers, but not far away either.

So I'm removing this and the docs in MDN so we can start fresh.

BCD PR is mdn/browser-compat-data#23989

Note, there was a previous attempt to do this in #15500 (which I blocked, but I was so very wrong :-) ).

@hamishwillee hamishwillee requested review from a team as code owners July 30, 2024 07:46
@hamishwillee hamishwillee requested review from pepelsbey and sideshowbarker and removed request for a team July 30, 2024 07:46
@github-actions github-actions bot added Content:WebAPI Web API docs Content:JS JavaScript docs Content:Firefox Content in the Mozilla/Firefox subtree labels Jul 30, 2024
@github-actions github-actions bot added the size/m [PR only] 51-500 LoC changed label Jul 30, 2024
Copy link
Contributor

Preview URLs

Flaws (1)

Note! 2 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/Element
Title: Element
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/Element/getBoxQuads does not exist
External URLs (77)

URL: /en-US/docs/Web/API/Element
Title: Element


URL: /en-US/docs/Mozilla/Firefox/Experimental_features
Title: Experimental features in Firefox

@sideshowbarker sideshowbarker merged commit 041d9aa into mdn:main Jul 30, 2024
8 checks passed
@hamishwillee hamishwillee deleted the delete_html_sanitizer_api branch July 30, 2024 21:35
@hamishwillee
Copy link
Collaborator Author

FYI @lukewarlow

@lukewarlow
Copy link
Contributor

Glad to see it! :) I definitely think this is for the best.

yin1999 added a commit to mdn/translated-content that referenced this pull request Jul 31, 2024
yin1999 added a commit to mdn/translated-content that referenced this pull request Jul 31, 2024
* zh-cn: sync translated content

* delete HTML Sanitizer API

ref: mdn/content#35238

---------

Co-authored-by: Allo <yin199909@aliyun.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Firefox Content in the Mozilla/Firefox subtree Content:JS JavaScript docs Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants