[Konsole-devel] Review Request 108844: Add support for ANSI SGR escape code 3 (italics) to Konsole

[Commit Hook]

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


This review has been submitted with commit 68a98ed77063e622985d422b625d7dc5895f10c3 by Kurt Hindenburg to branch master.

- Commit Hook


On Feb. 8, 2013, 12:29 a.m., Jem Orgun wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108844/
> -----------------------------------------------------------
> 
> (Updated Feb. 8, 2013, 12:29 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> Adds support for escape sequences \e[3m and \e[23m (to turn italics on and off respectively). I have no idea if this is actually a standard, but it's listed on Wikipedia, (http://en.wikipedia.org/wiki/ANSI_escape_code#CSI_codes), implemented in urxvt, and used by vim, so evidently it has some grounding.
> This is my first time doing this, so if I'm doing something wrong please tell me.
> 
> 
> Diffs
> -----
> 
>   src/Character.h ced1621 
>   src/TerminalDisplay.cpp d14daba 
>   src/Vt102Emulation.cpp 1ab7758 
> 
> Diff: http://git.reviewboard.kde.org/r/108844/diff/
> 
> 
> Testing
> -------
> 
> ` echo -e "\e[1mbold\e[22m \e[3mitalic\e[23m \e[4munderline\e[24m \e[5mblink\e[25m \e[7mreverse\e[27m" ' works as expected.
> 
> 
> Thanks,
> 
> Jem Orgun
> 
>

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