Re: [O] [New Exporter] Parameterized wrapper elements

2013-03-09 Thread Jambunathan K
Rick I have my reservations in applying this patch - I am not concerned about the patch, I have not looked at it. Any improvements to existing backends should invariably answer the question - "Can this change improve export tools or the parse tree syntax." If such a question is never asked and

Re: [O] [New Exporter] Parameterized wrapper elements

2013-03-09 Thread Rick Frankel
On Sat, Mar 09, 2013 at 10:32:11AM +0100, Bastien wrote: > Hi Rick, > One thing you may double-check in the meantime is: is it > compatible with the org-info.js utility? The default should > be "yes", even if users can replace "div" by something else > (e.g. for the needs of specific backends.)

Re: [O] [New Exporter] Parameterized wrapper elements

2013-03-09 Thread Rick Frankel
On Sat, Mar 09, 2013 at 01:46:37AM +0100, Nicolas Goaziou wrote: > Since I don't use html back-end, it would be better to hear from actual > users what they think about it. Sorry, forgot that you are not the keeper of ox-html, just the new exporter at large ;). > Anyway, just a few comments: >

Re: [O] [New Exporter] Parameterized wrapper elements

2013-03-09 Thread Bastien
Hi Rick, besides Nicolas good suggestions regarding the code, I think the patch is good and I welcome more flexibility in the HTML exporter so that HTML5-ready derived backends can be written. I'll have a careful look next week. One thing you may double-check in the meantime is: is it compatible

Re: [O] [New Exporter] Parameterized wrapper elements

2013-03-08 Thread Jambunathan K
Rick Frankel writes: > (cc'ing list) > > Nicolas- > > The patch Jambunathan sent didn't really make much sense to me, That's fine. I didn't understand what you are saying then. Show me examples of how HTML5 code or deck code looks like. Show me your backend code (specifics of what function

Re: [O] [New Exporter] Parameterized wrapper elements

2013-03-08 Thread Nicolas Goaziou
Hello, Rick Frankel writes: > Anyway, attached is a patch which parameterizes the html exporter in a > way which is useful (for me :) in deriving new backends. It also make > the exporter more capable of generating HTML5 compatible output > instead of just XHTML. Thank you for your patch. > If

[O] [New Exporter] Parameterized wrapper elements

2013-03-08 Thread Rick Frankel
(cc'ing list) Nicolas- The patch Jambunathan sent didn't really make much sense to me, as it didn't provide any added functionality over simply subclassing (deriving) from the current html exporter. Anyway, attached is a patch which parameterizes the html exporter in a way which is useful (for m