From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Price Subject: Re: extra paragraph in image export? Date: Tue, 24 Jan 2017 23:04:48 -0500 Message-ID: References: <87lgu3b5c8.fsf@nicolasgoaziou.fr> <87sho8ztgv.fsf@gmx.us> <87fuk86tdg.fsf@nicolasgoaziou.fr> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=001a113ff00c78039c0546e35511 Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:44327) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cWEpG-0000lo-LS for emacs-orgmode@gnu.org; Tue, 24 Jan 2017 23:04:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cWEpF-0004k2-Ou for emacs-orgmode@gnu.org; Tue, 24 Jan 2017 23:04:50 -0500 Received: from mail-io0-x229.google.com ([2607:f8b0:4001:c06::229]:35613) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1cWEpF-0004ju-Kg for emacs-orgmode@gnu.org; Tue, 24 Jan 2017 23:04:49 -0500 Received: by mail-io0-x229.google.com with SMTP id j18so4999858ioe.2 for ; Tue, 24 Jan 2017 20:04:49 -0800 (PST) In-Reply-To: <87fuk86tdg.fsf@nicolasgoaziou.fr> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: Nicolas Goaziou Cc: Org Mode , Rasmus --001a113ff00c78039c0546e35511 Content-Type: text/plain; charset=UTF-8 On Tue, Jan 24, 2017 at 4:27 PM, Nicolas Goaziou wrote: > Hello, > > Rasmus 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

%s

" contents) + (format "\n%s" contents) Because we never want the

tag inside the figure. Isn't that right? --001a113ff00c78039c0546e35511 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable



On Tue, Jan 24, 2017 at 4:27 PM, Nicolas Goaziou <= mail@nicolasgoa= ziou.fr> wrote:
Hello,

Rasmus <rasmus@gmx.us> writes:
> I'm not an expert of HTML either.=C2=A0 However, the "p"= tag inside the
> "figure" is uncommon looking a bit on the example usages onl= ine.=C2=A0 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 n= ot
> alike.=C2=A0 I.e. the former is "figure>img" the other is= a "p>img".=C2=A0 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 lik= e it to me -- I would instead have put:

-=C2=A0=C2=A0 =C2=A0=C2=A0= =C2=A0=C2=A0 (format "\n<p>%s</p>" contents)
+=C2= =A0=C2=A0 =C2=A0=C2=A0=C2=A0=C2=A0 (format "\n%s" contents)
Because we never want the <p> tag inside the figure. Isn= 't that right?=C2=A0

--001a113ff00c78039c0546e35511--