github: shadowy octopus with the head of a robot, emblazoned with the Dreamwidth swirl (Default)
github ([personal profile] github) wrote in [site community profile] changelog2020-11-30 11:59 pm

[dreamwidth/dw-free] 81b7dd: [#2842] Stop closing `` placeholder tags

Branch: refs/heads/master Home: https://github.com/dreamwidth/dw-free Commit: 81b7ddeb3ae53cb22a91edf77e1a959f3f266f03 https://github.com/dreamwidth/dw-free/commit/81b7ddeb3ae53cb22a91edf77e1a959f3f266f03 Author: Nick Fagerlund nick.fagerlund@gmail.com Date: 2020-11-30 (Mon, 30 Nov 2020)

Changed paths: M cgi-bin/LJ/CleanHTML.pm

Log Message:


[#2842] Stop closing <poll-45123> placeholder tags

The HTML cleaner closes dangling tags (both at document end, and when closing tags mid-way), but exempts the HTML5 list of "empty" tags from that.

Once an entry with a poll is saved, the markup that was used to build the poll gets replaced with a <poll-nnn> placeholder, so the poll can't be subsequently modified. These placeholder tags are effectively empty tags, but the cleaner doesn't really treat them as special because it's not in charge of rendering them; LJ::Poll->expand_entry does that, and we call it after clean_event when we're rendering an entry.

Anyway, the cleaner was helpfully "closing" these placeholders, so, let's don't.


Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
No Subject Icon Selected
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org