From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Abrahamsen Subject: Re: [PATCH] export to various flavors of (X)HTML Date: Sat, 20 Apr 2013 10:59:32 +0800 Message-ID: <87y5cdkiuj.fsf@ericabrahamsen.net> References: <878v4elu5h.fsf@ericabrahamsen.net> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([208.118.235.92]:44641) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UTNvl-0006B3-BU for emacs-orgmode@gnu.org; Fri, 19 Apr 2013 22:53:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UTNvk-0000oS-9n for emacs-orgmode@gnu.org; Fri, 19 Apr 2013 22:53:37 -0400 Received: from plane.gmane.org ([80.91.229.3]:42112) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UTNvk-0000oH-2p for emacs-orgmode@gnu.org; Fri, 19 Apr 2013 22:53:36 -0400 Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1UTNvg-0002Fm-Rn for emacs-orgmode@gnu.org; Sat, 20 Apr 2013 04:53:32 +0200 Received: from 221.216.167.68 ([221.216.167.68]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sat, 20 Apr 2013 04:53:32 +0200 Received: from eric by 221.216.167.68 with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sat, 20 Apr 2013 04:53:32 +0200 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-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: emacs-orgmode@gnu.org Rick Frankel writes: > On 19.04.2013 05:57, Eric Abrahamsen wrote: >> I'm starting a new thread for this since the previous discussion was >> buried in with something tangential. > >> I'm not proud of some of the implementation (self-closing vs >> non-self-closing tags are ugly, and I wish org-html-html5-p and >> org-html-xhtml-p were variables, not functions), but there it is, it >> seems to work. >> >> If this is deemed okay I'll send a version of the patch with a proper >> commit message, and also updated documentation. Thanks for looking at this! > I disagree with the minimized closing patch change. > > All versions of html accept the " />" idiom (with the extra space so > that html4 only browsers > don't break) for minimized tags (also ">" is, > i believe, always valid). html5 certainly accepts valid xhtml as > input. It would entirely break e.g, nxml-mode or xsl post-processing > to make this change. The " />" style doesn't validate for html4, that's what I was going on. It certainly doesn't make my browser explode, but I wanted that little green checkmark! If we can live with that, that's fine, or I can try to come up with a less hacky way of handling closing tags -- a macro maybe. > Other things that don't need to be removed for html5: > > - CDATA escapes > - xmns: .. xml:lang declarations (as long as you keep the html > valid xml) I'd be happy to leave the CDATA escapes in there, since it really doesn't seem to make any difference, and the implementation is ugly. I think I'm erring on the side of pedantic correctness. The xmns declaration, on the other hand, seems quite meaningless for anything that isn't xhtml (even if it doesn't actually break), and it's only a couple of lines of code to deal with, I'd rather keep that in there... Thanks again, Eric > > As a positive side effect, backing out these changes would simplify > the patch a lot :) > > > The doctype (and fix to the text/javascript closing tag) changes look > great. > > rick