←back to thread

Element: setHTML() method

(developer.mozilla.org)
167 points todsacerdoti | 1 comments | 22 Oct 25 09:03 UTC | HN request time: 0.198s | source
Show context
AlienRobot ◴[22 Oct 25 21:07 UTC] No.45675157[source]▶
Great functionality, terrible name.
replies(3): >>45675224 #>>45675738 #>>45678913 #
jonathrg ◴[22 Oct 25 22:05 UTC] No.45675738[source]▶
Why? Does it not set the HTML?
replies(1): >>45676209 #
netsharc ◴[22 Oct 25 22:57 UTC] No.45676209[source]▶
It doesn't say "There's a lot of hidden sanitizing stuff inside this method" from the name...

Something like "setSafeHTML()" would be preferable. (Since it's Mozilla, there should be a few committee meetings to come up with the appropriate name)...

replies(1): >>45677148 #
1. hoppp ◴[23 Oct 25 01:21 UTC] No.45677148[source]▶
Well ,could it be safelySetHTML instead of setSafeHTML ?

The second one could imply the HTML is already safe while the first one is safe way to set html.

If it's just setHTML then it could imply that don't care if its safe or not.