From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org>
Subject: Re: Org-special-blogs does not make well-formed xhtml
Date: Mon, 03 Oct 2011 15:26:19 +0200
Message-ID: <80r52unr44.fsf@somewhere.org>
References: <4E89AE38.3020309@christianmoe.com>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org-mXXj517/zsQ@public.gmane.org>
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request-mXXj517/zsQ@public.gmane.org?subject=unsubscribe>
List-Archive: </archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode-mXXj517/zsQ@public.gmane.org>
List-Help: <mailto:emacs-orgmode-request-mXXj517/zsQ@public.gmane.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request-mXXj517/zsQ@public.gmane.org?subject=subscribe>
Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org-mXXj517/zsQ@public.gmane.org
Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org-mXXj517/zsQ@public.gmane.org
To: emacs-orgmode-mXXj517/zsQ@public.gmane.org

Hi Christian Moe,

Christian Moe wrote:
> Hi,
>
> XHTML produced with Org-special-blocks is not well-formed; <div> tags 
> get wrapped in <p> tags. Example:
>
>     Some text.
>
>     #+begin_sidebar
>       Some details left out of the main text.
>
>       Some more details.
>     #+end_sidebar
>
>     Some more text.
>
> This results in the following html, which causes XML processors to fail.
>
>     <p>
>     <div class="sidebar">
>       Some details left out of the main text.
>     </p>
>     <p>
>       Some more details.
>     </div>
>     </p>
>     <p>
>     Some more text.
>     </p></div>
>
> The problem seems to be fixed by un-commenting the fifth line in the 
> below function in org-special-blocks.el:
>
>     (defun org-special-blocks-convert-html-special-cookies ()
>       "Converts the special cookies into div blocks."
>       ;; Uses the dynamically-bound variable `line'.
>       (when (string-match "^ORG-\\(.*\\)-\\(START\\|END\\)$" line)
>     ;    (org-close-par-maybe)
>         (message "%s" (match-string 1))
>         (if (equal (match-string 2 line) "START")
> 	   (insert "<div class=\"" (match-string 1 line) "\">\n")
> 	 (insert "</div>\n"))
>         (throw 'nextline nil)))
>
> Does anyone know if that was commented out for a reason?

git blame suggests it was there since that file org-special-blocks.el has been
added (in contrib/lisp, by Carsten, on 2009-05-07 13:53)...

Best regards,
  Seb

-- 
Sebastien Vauban