On Tue, Jan 24, 2017 at 4:27 PM, Nicolas Goaziou <m...@nicolasgoaziou.fr> wrote:
> Hello, > > Rasmus <ras...@gmx.us> writes: > > > I'm not an expert of HTML either. However, the "p" tag inside the > > "figure" is uncommon looking a bit on the example usages online. Thus, I > > agree with Matt that it should probably not be there. > > OK. > > > AFAIK, the CSS selector still differs so on the output side they are not > > alike. I.e. the former is "figure>img" the other is a "p>img". Aside: > If > > anything, removing the "p" tag inside the figure would make it easier to > > handle images. > > So would the following patch fix the issue? > > I'm not sure. Doesn't look like it to me -- I would instead have put: - (format "\n<p>%s</p>" contents) + (format "\n%s" contents) Because we never want the <p> tag inside the figure. Isn't that right?
