Please do not re-implement HTML sanitization. Just replace that escapeForHTML an... | Hacker News

Hacker Newsnew | past | comments | ask | show | jobs | submit

randomfool on May 7, 2022 | parent | context | favorite | on: TodoMVC App Written in Vanilla JavaScript

Please do not re-implement HTML sanitization. Just replace that escapeForHTML and innerHTML assignment with a later textContent assignment. Please. Pretty please.

grabanski on May 7, 2022 [–]

That seems like a good improvement. Pull requests welcome!

grabanski on May 7, 2022 | [–]

How does this look? https://github.com/1Marc/todomvc-vanillajs-2022/commit/d38e5...

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact