[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [patch][ox-html] Support for level based containers
From: |
Rick Frankel |
Subject: |
Re: [O] [patch][ox-html] Support for level based containers |
Date: |
Mon, 17 Mar 2014 20:33:39 -0400 |
User-agent: |
Mutt/1.5.22 (2013-10-16) |
On Mon, Mar 17, 2014 at 11:26:28PM +0100, Rasmus wrote:
> Rick Frankel <address@hidden> writes:
>
> > On Mon, Mar 17, 2014 at 03:15:50AM +0100, Bastien wrote:
> >> Hi Rasmus and Nicolas,
> >>
> >
> >> Nicolas Goaziou <address@hidden> writes:
> >>
> >> >> Let me know if you find you'd be willing to merge something like this
> >> >
> >> > I don't know enough HTML to have an opinion here.
> >>
> >
> > I don't think it's a bad change, but i have a couple of concerns:
> >
> > 1. It's a breaking change.
> > 2. The default should mimic the current functionality:
> > ='((div . div))=
> > 3. The customization should be more structured, not just a
> > string (see e.g., =org-html-text-markup-alist=.)
> >
> > Attached is a modification of the patch which fixes 2 & 3. #1 is a
> > question more for Nicolas & Bastien...
>
> It seems you modified v1 rather than v2 that I send last night¹? I'm
> not sure if point 1 holds in the second revision as it explicitly
> allows for a string as now. I think I disagree with your point 2 as it
> only shows up when you use HTML5-fancy. On point 3,
But not everyone using html5-fancy would agree with your choice of
hierarchy, and it should be up to indvidual users to add additional
semantic structure to the output.
> org-html-text-markup-alist is nice. What do you want to see in
> addition to the current structure (in patch v2)?
Somehow I never saw the original thread, only the email cc'ing me
directly. I went to gmane to find the patch, and obviously grabbed the
wrong one.
Could you please send me the (new) patch so that i can review it?
thanx,
rick
- [O] [patch][ox-html] Support for level based containers, Rasmus, 2014/03/15
- Re: [O] [patch][ox-html] Support for level based containers, Nicolas Goaziou, 2014/03/16
- Re: [O] [patch][ox-html] Support for level based containers, Rasmus, 2014/03/16
- Re: [O] [patch][ox-html] Support for level based containers, Bastien, 2014/03/16
- Re: [O] [patch][ox-html] Support for level based containers, Rick Frankel, 2014/03/17
- Re: [O] [patch][ox-html] Support for level based containers, Rasmus, 2014/03/17
- Re: [O] [patch][ox-html] Support for level based containers,
Rick Frankel <=
- Re: [O] [patch][ox-html] Support for level based containers, Rasmus, 2014/03/17
- Re: [O] [patch][ox-html] Support for level based containers, Rick Frankel, 2014/03/18
- Re: [O] [patch][ox-html] Support for level based containers, Rasmus, 2014/03/18
- Re: [O] [patch][ox-html] Support for level based containers, Rick Frankel, 2014/03/19