Escape codes SGR 2, SGR 8, SGR 9, and SGR 53 are not implemented in Konsole. All except SGR 53 are implemented in xterm. Reproducible: Always Steps to Reproduce: echo -e 'D\e[2mD\e[9mD\e[53mD\e[8mD' Actual Results: 5 D symbols, all identical Expected Results: 4 D symbols, the last three with "dim," "faint" or "half" intensity, the last two with "strikeout" font, and the last one overlined. The final, fifth D should not be visible. I posted an earlier, less-complete patch set to the development mailing list. I don't know if that was the correct venue---or if this one is either.
Created attachment 98546 [details] SGR Implementation
What is this attachment? gz or bz2?
The attachment should be a tar.gz with 7 patches. On my debian machine it extracts with tar -xf . I've been using a version of konsole with these patches applied for about a month now, and haven't hit any bugs.
Even w/ all the patches, I only get 4 D - does it work for you w/ these patches? What color scheme to you use?
The patch depends on a color style to implement the "faint" intensities for each color. I did this specifically for Linux.colorscheme (and it should is included in the patch) but did not get around to the other themes. My eye is not particularly suited for aesthetic choices, and the choice of faint colors for the themes is such a choice (in particular the solarized and light themes seemed to require some thought). Assuming this bug gets enough attention to be included, but not enough for other people with better aesthetics than me to work on the themes, I'll hack up some faint colors. I'll point out that the patch also includes in a fully functional (but completely untested) gui for setting the faint color theme---right next to setting the other colors and intense colors. Notwithstanding that, the overline and strikeout font effects should function correctly. Are you seeing those with the patch applied? If not, could you confirm that your terminal font supports overline and strikeout effects?
Created attachment 99927 [details] Revised complete patchset tar.gz of 6 patches
The revised patchset has support for all default color schemes. There should be no issues using these color schemes.
Hi! https://git.reviewboard.kde.org/dashboard/ is the preferred method of submitting patches for Konsole.
The patches works for me, I get the correct strikethrough/overline, and the last one is invisible. The first one is with the normal color, the three remaining are in the faint color defined in the color scheme. However it seems like neither urxvt nor uxterm here handles it correctly.
I am glad that the patch works for others. Sorry if this is turning into spam. I made a review request https://git.reviewboard.kde.org/r/128390/ but since I have a series of patches that depend on the previous one, I was not sure how to upload the sequence without squashing all the commits. What should I now do? Wait for someone to review my draft proposal? Thanks, Antonio
Reviewboard isn't very good with series of patches, though you can mark patches as dependent on others I haven't really figured out the details. FWIW I don't seem to have access to that review request, but I guess hindenburg has?
I really appreciate your getting back to me. Thank you. I cannot "publish" the patch without it having a "review"---and I don't think I am allowed to review it myself (?). The first step in the patch creation process required uploading a diff---which for subsequent patches is rejected because the parent revision is not yet known to reviewboard (or, it was not properly detected). I think. Hindenburg glanced at this bug a couple months ago. Could you possibly contact him, or should I maybe ping him directly?
He is very active reviewing patches, he'll probably get to this very soon.
Git commit 84b43dfb2108eab47fa1dfcafbf1b94a410d6cbf by Kurt Hindenburg. Committed on 16/07/2016 at 16:34. Pushed by hindenburg into branch 'master'. Add rendition flags SGRs 2, 8, 9, 53 Adds faint intensity, strikeout, conceal and overline support. echo -e 'D\e[2mD\e[9mD\e[53mD\e[8mD' Thanks to Antonio Russo antonio e russo gmail com for patch REVIEW: 128405 M +30 -0 data/color-schemes/BlackOnLightYellow.colorscheme M +30 -0 data/color-schemes/BlackOnRandomLight.colorscheme M +30 -0 data/color-schemes/BlackOnWhite.colorscheme M +30 -0 data/color-schemes/BlueOnBlack.colorscheme M +30 -0 data/color-schemes/Breeze.colorscheme M +30 -0 data/color-schemes/DarkPastels.colorscheme M +30 -0 data/color-schemes/GreenOnBlack.colorscheme M +30 -0 data/color-schemes/Linux.colorscheme M +30 -0 data/color-schemes/RedOnBlack.colorscheme M +30 -0 data/color-schemes/Solarized.colorscheme M +30 -0 data/color-schemes/SolarizedLight.colorscheme M +30 -0 data/color-schemes/WhiteOnBlack.colorscheme M +16 -11 src/Character.h M +22 -4 src/CharacterColor.h M +40 -3 src/ColorScheme.cpp M +19 -5 src/ColorSchemeEditor.cpp M +1 -1 src/History.h M +25 -33 src/Screen.cpp M +6 -5 src/Screen.h M +1 -1 src/TerminalCharacterDecoder.h M +13 -3 src/TerminalDisplay.cpp M +10 -1 src/Vt102Emulation.cpp M +13 -1 src/autotests/CharacterColorTest.cpp http://commits.kde.org/konsole/84b43dfb2108eab47fa1dfcafbf1b94a410d6cbf
Git commit c8da8f3b4745c9bd7ef24890a15457342d2ed860 by Kurt Hindenburg. Committed on 16/07/2016 at 17:40. Pushed by hindenburg into branch 'Applications/16.08'. Add rendition flags SGRs 2, 8, 9, 53 Adds faint intensity, strikeout, conceal and overline support. echo -e 'D\e[2mD\e[9mD\e[53mD\e[8mD' Thanks to Antonio Russo antonio e russo gmail com for patch REVIEW: 128405 (cherry picked from commit 84b43dfb2108eab47fa1dfcafbf1b94a410d6cbf) M +30 -0 data/color-schemes/BlackOnLightYellow.colorscheme M +30 -0 data/color-schemes/BlackOnRandomLight.colorscheme M +30 -0 data/color-schemes/BlackOnWhite.colorscheme M +30 -0 data/color-schemes/BlueOnBlack.colorscheme M +30 -0 data/color-schemes/Breeze.colorscheme M +30 -0 data/color-schemes/DarkPastels.colorscheme M +30 -0 data/color-schemes/GreenOnBlack.colorscheme M +30 -0 data/color-schemes/Linux.colorscheme M +30 -0 data/color-schemes/RedOnBlack.colorscheme M +30 -0 data/color-schemes/Solarized.colorscheme M +30 -0 data/color-schemes/SolarizedLight.colorscheme M +30 -0 data/color-schemes/WhiteOnBlack.colorscheme M +16 -11 src/Character.h M +22 -4 src/CharacterColor.h M +40 -3 src/ColorScheme.cpp M +19 -5 src/ColorSchemeEditor.cpp M +1 -1 src/History.h M +25 -33 src/Screen.cpp M +6 -5 src/Screen.h M +1 -1 src/TerminalCharacterDecoder.h M +13 -3 src/TerminalDisplay.cpp M +10 -1 src/Vt102Emulation.cpp M +13 -1 src/autotests/CharacterColorTest.cpp http://commits.kde.org/konsole/c8da8f3b4745c9bd7ef24890a15457342d2ed860