pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter_e@gmx.net>
To: Josh Kupershmidt <schmiddy@gmail.com>
Cc: pgsql-docs@postgresql.org
Subject: Re: Large SGML Cleanup
Date: Wed, 03 Nov 2010 16:24:38 +0200
Message-ID: <1288794278.20884.2.camel@vanquo.pezone.net> (raw)
In-Reply-To: <AANLkTi=1Sm9N3Khiued9UiMfdd_TKLimMiO9mCfHtL39@mail.gmail.com>
References: <AANLkTi=1Sm9N3Khiued9UiMfdd_TKLimMiO9mCfHtL39@mail.gmail.com>

On tis, 2010-11-02 at 22:56 -0400, Josh Kupershmidt wrote:
> This HTML fails validation, as one isn't supposed to be stuffing
> tables inside <p> nodes. The attached patch fixes all the instances of
> this I could find, by closing out <para> nodes before beginning lists
> and tables.

Um, this is like moving around the C code because the compiler generates
invalid assembly code.  Fix the compiler.

That said, we have the following in stylesheet.dsl:

;; Block elements are allowed in PARA in DocBook, but not in P in
;; HTML.  With %fix-para-wrappers% turned on, the stylesheets attempt
;; to avoid putting block elements in HTML P tags by outputting
;; additional end/begin P pairs around them.
(define %fix-para-wrappers% #t)

So evidently someone thought of this before and put something in to
prevent some/many/most cases.

In general, I think the more efficient way to address this overall
problem is to run the resulting HTML through tidy and be done with it.




view thread (19+ messages)  latest in thread

Message-ID: <1288794278.20884.2.camel@vanquo.pezone.net>
Permalink:  ../1288794278.20884.2.camel@vanquo.pezone.net/
Also on:    postgresql.org/message-id/1288794278.20884.2.camel@vanquo.pezone.net

 · 

reply

Reply instructions:

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

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-docs@postgresql.org
  Cc: peter_e@gmx.net, schmiddy@gmail.com
  Subject: Re: Large SGML Cleanup
  In-Reply-To: <1288794278.20884.2.camel@vanquo.pezone.net>

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

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox