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

Use sanitizeFor *and* setHTML #5

Open
mozfreddyb opened this issue Dec 15, 2021 · 1 comment
Open

Use sanitizeFor *and* setHTML #5

mozfreddyb opened this issue Dec 15, 2021 · 1 comment

Comments

@mozfreddyb
Copy link

Daniel was looking at the usercounters for the API and the (very low) numbers are in favor of sanitizeFor. I'd like to argue that people use pages like this one to test & learn about the API and that is likely skewing the numbers.

I'll send a tiny PR that makes sure the page uses both if that's OK with y'all :)

@mikewest
Copy link
Owner

I don't think Chrome's UseCounter numbers at the moment will tell you anything useful: we haven't shipped the API, so the set of users poking at it is tiny and entirely non-representative.

That said, if you'd like to add setHTML to the playground, go for it! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants