Bug 366143

Summary:	Gray color is not visible in 'solarized' theme
Product:	[Applications] konsole	Reporter:	Simon Huerlimann <simon.huerlimann>
Component:	general	Assignee:	Konsole Developer <konsole-devel>
Status:	RESOLVED INTENTIONAL
Severity:	normal	CC:	marcorichetta
Priority:	NOR	Keywords:	investigated, triaged
Version:	unspecified
Target Milestone:	---
Platform:	Kubuntu
OS:	Linux
URL:	https://github.com/mattbrictson/airbrussh/issues/84
Latest Commit:		Version Fixed In:
Attachments:	Non-visible Text in solarized Visible Text in solarized light Breeze theme Solarized theme

Description Simon Huerlimann 2016-07-26 21:16:58 UTC

When using the 'solarized' theme, text output that uses the ANSI escapes for gray is not visible. it seems to use the same color as the background color.

It is okay with all other themes, including 'solarized light'.

It seem to be the same on "'Solarized' Color Scheme on Ubuntu Gnome 16.04" so the issue might be in some share upstream code.

Please see https://github.com/mattbrictson/airbrussh/issues/84 for some screenshots and more information.

Reproducible: Always

Comment 1 Simon Huerlimann 2016-07-26 21:17:45 UTC

Created attachment 100321 [details]
Non-visible Text in solarized

Comment 2 Simon Huerlimann 2016-07-26 21:18:12 UTC

Created attachment 100322 [details]
Visible Text in solarized light

Comment 3 Marco Richetta 2018-10-30 20:14:04 UTC

Hi, I'm from the Bugsquad team and we are triaging bugs on Konsole today.
Thanks for reporting the bug.

Application Version: 17.12.3
KDE Plasma Version: 5.12.6
KDE Frameworks Version: 5.44.0
Qt Version: 5.9.5

I took this steps to reproduce:

1- On Konsole configure the theme to "Solarized". (The dark solarized)
2- Launch python3 on Konsole.
3- Based on this test on SO (https://stackoverflow.com/a/39452138/6389248), enter the code below.

> CRED = '\33[90m'
> CEND = '\33[0m'
> print(CRED + "HELLO" + CEND)

4- It should print an empty line.
5- Now configure Konsole to Breeze theme.
6- The message now should be seen.

7- On the same session on Breeze theme, update the CRED value. 
> CRED = '\33[30m'
> print(CRED + "HELLO" + CEND)

8- The new message is invisible on that theme.
9- Change to solarized theme and you'll see the message barely visible.

Conclusion

I can reproduce the bug but I think it's inevitable to happen on any theme, i.e. grey characters on the same grey background will never be seen.

I'll change the status to INTENTIONAL because I think the behaviour of the application is as expected.

Comment 4 Marco Richetta 2018-10-30 20:15:12 UTC

Created attachment 115993 [details]
Breeze theme

Comment 5 Marco Richetta 2018-10-30 20:15:34 UTC

Created attachment 115994 [details]
Solarized theme