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.