Re: [O] [patch][ox-latex] context-aware subscript

2013-08-31 Thread Nicolas Goaziou
Rasmus writes: > What I do in the patch is that I apply the filters within the > function Why is that? `org-export-data' already takes care of filters. These calls are redundant. > Also, filters using a regexp with "\text{whatever}" wouldn't work. I don't get it. >>> + (scripts '(subscript

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-31 Thread Rasmus
Hi, > Thanks for the patch. Here are a few comments. Thanks for the comments. >> filter replace all & $\beta_{\text{}}$ & \texttimes{}$_{\text{}}$ \\ > > We might be able to handle it more nicely, i.e. by skipping \text when > contents are empty. Right, these should be removed due to spac

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-31 Thread Nicolas Goaziou
Hello, Rasmus writes: > I've just enclosed a quick patch (as in doesn't contain proper commit > msg), but it's basically the previous patch minus the removal of \text > in math plus some quick checks towards potential nasty filters. > > It works with the following test file, but let me know abou

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-29 Thread Rasmus
Nicolas Goaziou writes: > Rasmus writes: > >>> Correct. Then, fixing it is more important than caring about some user >>> filter. >> >> OK, can I help? > > Sure, please go ahead. I've just enclosed a quick patch (as in doesn't contain proper commit msg), but it's basically the previous patch mi

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-28 Thread Nicolas Goaziou
Rasmus writes: >> Correct. Then, fixing it is more important than caring about some user >> filter. > > OK, can I help? Sure, please go ahead. > That's what I use as well, but I'd like to save the $'s around > constructs like \(\beta_t\). I'll try to fix it via a filter. A good filter could b

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-28 Thread Nicolas Goaziou
Hello, Rasmus writes: > I know it's intended but I dislike it. In earlier version of > ox-latex.el it didn't use the \text-macro. Earlier versions were broken in many ways. They didn't handle spaces, unicode characters and nested sub/superscript like \text does. >> Also, merging consecutive s

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-28 Thread Rasmus
Hi Nicolas, Nicolas Goaziou writes: >> I know it's intended but I dislike it. In earlier version of >> ox-latex.el it didn't use the \text-macro. > > Earlier versions were broken in many ways. They didn't handle spaces, > unicode characters and nested sub/superscript like \text does. OK. >>

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-28 Thread Rasmus
Hi Nicolas, Nicolas Goaziou writes: > Thanks for the patch. Here are some comments about it. Sorry about the late reply > Rasmus writes: > >> Currently one can't write something like \beta_t and get a nice >> result >> in org when exporting to LaTeX (where nice result := $\beta_t$). >> This >

Re: [O] [patch][ox-latex] context-aware subscript

2013-08-19 Thread Nicolas Goaziou
Hello, Thanks for the patch. Here are some comments about it. Rasmus writes: > Currently one can't write something like \beta_t and get a nice result > in org when exporting to LaTeX (where nice result := $\beta_t$). This > patch tries to fix it. Translating \beta_t into $\beta$$_\text{t}$ an