Bug 431216 - [REGRESSION][BISECTED] Text is barely visible due to white becoming gray
Summary: [REGRESSION][BISECTED] Text is barely visible due to white becoming gray
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: font (show other bugs)
Version: 20.12.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-01-06 10:30 UTC by Konstantin Kharlamov
Modified: 2021-01-06 21:17 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Konstantin Kharlamov 2021-01-06 10:30:50 UTC
SUMMARY

After update "20.08.3 → 20.12.0" all colors become darker. It would not be bad per se, except that white text become gray. It is best seen with a screenshot: https://i.imgur.com/ggsWDnn.png on the left is 20.08.3 and on the right is 20.12.0. The one on the right is unreadable.

STEPS TO REPRODUCE
1. Open konsole 
2. Look at diff through diffr utility.

OBSERVED RESULT

White text is all gray and is barely visible

EXPECTED RESULT

White text is white

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.20.4
KDE Frameworks Version: 5.77.0
Qt Version: 5.15.2
Comment 1 Konstantin Kharlamov 2021-01-06 10:55:13 UTC
Successfully bisected to commit:

    82806a2c129f8fa4b21be25b4108eecd8d460625 is the first bad commit
    commit 82806a2c129f8fa4b21be25b4108eecd8d460625
    Author: Carlos Alves <cbc.alves@gmail.com>
    Date:   Fri Nov 27 08:50:43 2020 -0300

        Fix bold character color paint

        Konsole was painting 'bold' character as 'faint bold' character.

        BUG: 405345
        FIXED-IN: 20.12
        (cherry picked from commit 270d6ea3247bb41a51535129e4b1c8eef51cf316)

     src/Screen.cpp | 8 +-------
     1 file changed, 1 insertion(+), 7 deletions(-)

I confirm that reverting the commit fixes the issue.

CCing the author.
Comment 3 Konstantin Kharlamov 2021-01-06 21:17:21 UTC
(In reply to Carlos Alves from comment #2)
> It was reverted:
> https://invent.kde.org/utilities/konsole/-/commit/
> 2d58ed02813f409ab0d140fcf4452643ea73ff2d
> 
> And pushed the revert to released branch:
> https://invent.kde.org/utilities/konsole/-/commit/
> 08aa65e32427f8dc49fbfd6b1d8997431b72dab2
> 
> A few weeks ago.

Thank you! I see, closing then as fixed