[Konsole-devel] Review Request 112507: Fix for “ HTML export has too many closing span tags in it”

[Kurt Hindenburg]

> On Sept. 4, 2013, 1:03 p.m., Kurt Hindenburg wrote:
> > Thanks it looks good.  On another topic, I wonder if it should be valid HTML w/ doctype/head/body/etc.

actually looking at the code more closely, can the " = false" just be put in closeSpan()?


- Kurt


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


On Sept. 4, 2013, 10:37 a.m., Martin Ueding wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112507/
> -----------------------------------------------------------
> 
> (Updated Sept. 4, 2013, 10:37 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> This fixes the linked bug.
> 
> 
> This addresses bug 324476.
>     http://bugs.kde.org/show_bug.cgi?id=324476
> 
> 
> Diffs
> -----
> 
>   src/TerminalCharacterDecoder.cpp 0615d5d 
> 
> Diff: http://git.reviewboard.kde.org/r/112507/diff/
> 
> 
> Testing
> -------
> 
> I built and tested it with three sessions, the exported HTML looked right.
> 
> 
> Thanks,
> 
> Martin Ueding
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20130904/f874a72f/attachment.html>