Differential D11706
Remarkup: Correctly render inline embed layout Authored by matmarex on Feb 6 2015, 10:56 PM. Tags Referenced Files
Subscribers
Details
The generated HTML is like <p>some text <div …>…</div> more text</p>, and HTML <p/> tags may not contain block content like <div/> tags. Browsers actually parse this as if it was <p>some text </p><div …>…</div> more text<p></p> (sic). The layout CSS class already has display: inline set, but this is not sufficient. Browser's HTML parser doesn't care what CSS rules will be applied, it only deals with the meanings of tags. Fixes T7201. Verify that the following displays the image inline: some text {Fnnn,layout=inline} more text
Diff Detail
Event TimelineComment Actions
Comment Actions It already implies display: inline in CSS, but as I have explained on the task that is not sufficient. Browser's HTML parser doesn't care what CSS rules will be applied, it only deals with the meanings of tags, and <p/> is specified as not being allowed to have a <div/> inside it. This is the same kind of thing as if you tried to nest <a/> tags, it's just not possible. |