emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Orgmode] [PATCH] Add support for multile html container classes.


From: David Maus
Subject: Re: [Orgmode] [PATCH] Add support for multile html container classes.
Date: Sun, 01 Aug 2010 10:58:03 +0200
User-agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (Gojō) APEL/10.8 Emacs/23.2 (i486-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO)

Bastien wrote:
>Hi David,

>David Maus <address@hidden> writes:

>> * org-html.el (org-html-level-start): Add multiple container
>> classes.

>I don't quite understand this patch.

>Can't we use

>  :PROPERTIES:
>  :HTML_CONTAINER_CLASS: class1 class2 class3
>  :END:

>already?

No: Org currently only uses the first class.  E.g.

,----
| (re-search-forward "^[ \t]*:HTML_CONTAINER_CLASS:[ \t]+\\(\\S-+\\)" nil t)
| (setq class (match-string 1))
`----

\\S-+ matches only the first sequence of non-space characters.

>PS: Also, please make sure the patch only reflect changes in the code
>(avoid refilling), it makes it easier to read ;)

Okay, I'll try to remember to provide separate patches for this.

Best,
  -- David
--
OpenPGP... 0x99ADB83B5A4478E6
Jabber.... address@hidden
Email..... address@hidden

Attachment: pgpsN2WNv8YLq.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]