emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Rick Frankel <rick@rickster.com>
To: emacs-orgmode@gnu.org
Subject: Re: [PATCH] export to various flavors of (X)HTML
Date: Fri, 19 Apr 2013 11:37:51 -0400	[thread overview]
Message-ID: <ed0b8ebcdddfee0e95f4e2ac53fdda17@mail.rickster.com> (raw)
In-Reply-To: <878v4elu5h.fsf@ericabrahamsen.net>

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.

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 "></{elem}", e.g. "<hr></hr>" 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.

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)


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

  reply	other threads:[~2013-04-19 15:38 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-19  9:57 [PATCH] export to various flavors of (X)HTML Eric Abrahamsen
2013-04-19 15:37 ` Rick Frankel [this message]
2013-04-20  2:59   ` Eric Abrahamsen
2013-04-23  1:00     ` Rick Frankel
2013-04-23  4:30       ` Eric Abrahamsen
2013-04-23  4:57         ` Samuel Wales
2013-04-23  6:55           ` Carsten Dominik
2013-04-23  7:54             ` Eric Abrahamsen
2013-04-23 12:09         ` François Pinard
2013-04-24  1:17           ` Christian Wittern
2013-04-24 13:10             ` François Pinard
2013-04-25 21:20               ` Eric Abrahamsen
2013-04-26 13:49                 ` Rick Frankel
2013-04-26 17:14                   ` Eric Abrahamsen
2013-04-26 18:49                     ` Rick Frankel
2013-04-29  6:02                       ` Eric Abrahamsen
2013-04-29  7:45                         ` Nicolas Goaziou
2013-04-30 14:38                         ` Rick Frankel
2013-04-30 14:40                         ` Rick Frankel
2013-05-01  3:26                           ` Eric Abrahamsen
2013-05-01 11:55                             ` Rick Frankel
2013-05-02 21:07                               ` Eric Abrahamsen
2013-05-03  7:17                                 ` Carsten Dominik
2013-05-06  5:49                                 ` Carsten Dominik
2013-05-06  7:36                                   ` Eric Abrahamsen
2013-05-06  7:48                                     ` Carsten Dominik
2013-05-06  9:05                                       ` Eric Abrahamsen
2013-05-06 12:56                                         ` Rick Frankel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ed0b8ebcdddfee0e95f4e2ac53fdda17@mail.rickster.com \
    --to=rick@rickster.com \
    --cc=emacs-orgmode@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).