Bug 483012

Summary: Changing line spacing for a Konsole profile causes misalignment until application restart
Product: [Applications] konsole Reporter: Luke Horwell <code>
Component: fontAssignee: Konsole Developer <konsole-devel>
Status: CONFIRMED ---    
Severity: normal CC: mobile.harvey, vasyl.demin
Priority: NOR Keywords: qt6
Version: 24.02.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Screenshot after changing line spacing to 16px
Cannot duplicate on Neon
Workng with Nano
My Konsole profile settings - working.
Still working with spacing at 16 px
Reproduced!

Description Luke Horwell 2024-03-09 17:55:44 UTC
Created attachment 166812 [details]
Screenshot after changing line spacing to 16px

SUMMARY

In Konsole 24.02.0, changing the line spacing for a profile causes the terminal to be misaligned (in some cases unreadable) until the application is restarted. This is also problematic if the user switches to a profile using different line spacing settings, unless it is the default profile.


STEPS TO REPRODUCE
1. Settings → Create/Edit a profile (use CTRL+ALT+M to show menu bars if hidden) 
2. Appearance → Miscellaneous
3. Set the line spacing to 8px.
4. Run an application like "nano" to observe text.

OBSERVED RESULT
The line spacing is broken, causing unreadable text, depending on the line spacing. This is a regression since Konsole 23.08.5 (Qt 5).

EXPECTED RESULT
Changing line spacing (via settings or switching profiles) render correctly without needing to restart Konsole.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma:  Arch Linux
KDE Plasma Version: 6.0.1
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2

ADDITIONAL INFORMATION
Tested under X11.

Observation for other users -- opening a new Konsole window (24.02.0, Qt 6) did seem smaller to its 23.08.5 (Qt 5) version after upgrading, then I found out line spacing wasn't applying properly and also needs a 1 pixel bump. I had it set to 1px up to now, but 2px makes it familiar again to how it was prior to upgrading (after restarting Konsole, of course!)

Appreciate the line spacing option, a nice feature for dyslexa users!
Comment 1 Nick 2024-03-16 12:23:05 UTC
Created attachment 167319 [details]
Cannot duplicate on Neon

Hi - I've tried this on Neon - Plasma 6.0.2 and Konsole 24.02.0 but it appears to work fine. See attached.

Can you please check if I'm doing anything wrong, or else we'll need to try to narrow this down somehow to reliably duplicate this one.

Thanks,
Nick
Comment 2 Nick 2024-03-16 12:23:59 UTC
Created attachment 167320 [details]
Workng with Nano
Comment 3 Nick 2024-03-16 12:24:36 UTC
Created attachment 167321 [details]
My Konsole profile settings - working.
Comment 4 Nick 2024-03-16 12:26:31 UTC
Created attachment 167322 [details]
Still working with spacing at 16 px
Comment 5 Nick 2024-03-16 13:50:23 UTC
Created attachment 167328 [details]
Reproduced!

OK - I wasn't reading the bug report correctly - apologies. I can now duplicate this on Neon as per the original description:

Open Konsole
Create a new profile with line spacing set to 16px
Then without closing Konsole, use the menu items to switch to this profile
The text is then all out of alignment.

To 'solve':
Close and re-open Konsole. However this step should not be required and sounds like a bug. Setting this as 'confirmed' hoping a dev can review further.
Comment 6 Nick 2024-03-16 13:52:27 UTC
Confirmed and duplicated on Neon.