Review Request 111501: Fix LaTeX filter

[Daniele E. Domenichelli]

> On July 14, 2013, 1:41 p.m., Martin Klapetek wrote:
> > filters/latex/latex-filter.cpp, line 76
> > <http://git.reviewboard.kde.org/r/111501/diff/1/?file=170559#file170559line76>
> >
> >     I'm not sure about this as it references something, that is not later..well, referenced. I mean the formula should have prepended "(%1)" as well I think. This will be weird especially with longer text - "this is the formula (1) text text text text text text text text text text text text text text text text text text text text text text text text text text text text text 
> >     
> >     LaTeX"
> >     
> >     Maybe it could be even replaced with "(formula below)"?

Usually you use [1], not (1) for citation, also I had something similar in the "code" plugin, so I think we could share a way to have references to the appended parts (perhaps have an link as well?)
Also the appended part should have the same number of the reference, or it will become confusing if you have several appended items in the same message


- Daniele E.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111501/#review35936
-----------------------------------------------------------


On July 14, 2013, 10:05 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111501/
> -----------------------------------------------------------
> 
> (Updated July 14, 2013, 10:05 a.m.)
> 
> 
> Review request for Telepathy and Andrea Scarpino.
> 
> 
> Description
> -------
> 
> Fix LaTeX filter
> 
> Instead of trying to put the images inline we put a reference and insert the LaTeX images at the end.
> 
> The "appended message parts" aren't affected by other filters.
> 
> 
> Diffs
> -----
> 
>   filters/latex/latex-filter.cpp 4e9601f095c1a3790e719b1cbbd1f32ed75c0424 
> 
> Diff: http://git.reviewboard.kde.org/r/111501/diff/
> 
> 
> Testing
> -------
> 
> currently code renders as <img src=".... because we run before the escape filter. 
> 
> If we run after we get bugs because we escape before the LaTeX can get the original code.
> 
> This fixes both of them, and makes it clearer for larger amounts of text, as otherwise line heights get confusing.
> 
> 
> File Attachments
> ----------------
> 
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/07/14/formula.png
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130714/8b1e8b7e/attachment.html>