Bug 446655

Summary: Text selection is broken if the selected text contains both English and non-English characters (character moves back and forth)
Product: [Applications] konsole Reporter: Fushan Wen <qydwhotmail>
Component: fontAssignee: Konsole Developer <konsole-devel>
Status: RESOLVED FIXED    
Severity: normal Keywords: regression
Priority: NOR    
Version: master   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: https://invent.kde.org/utilities/konsole/commit/f3d43ae3cf02988b49c67ec6057e19931baaeedc Version Fixed In:
Sentry Crash Report:

Description Fushan Wen 2021-12-08 01:57:54 UTC 
SUMMARY
The characters become jumpy if the selected text contains both English and non-English characters.


STEPS TO REPRODUCE
1. Run `echo "hello world 你好 世界"`
2. Try to select the output line

OBSERVED RESULT
Non-English characters move back and forth, making it difficult to control the selection.

EXPECTED RESULT
All characters remain steady in their original places.

SOFTWARE/OS VERSIONS
Information for package konsole:
--------------------------------
Repository     : unstable-kde-applications
Name           : konsole
Version        : 22.03.70git.20211203T171311~6138b5470-ku.68.1
Arch           : x86_64
Vendor         : obs://build.opensuse.org/KDE:Unstable
Installed Size : 615.3 KiB
Installed      : Yes
Status         : up-to-date
Source package : konsole-22.03.70git.20211203T171311~6138b5470-ku.68.1.src
Summary        : KDE Terminal
Comment 1 Fushan Wen 2021-12-08 02:53:47 UTC 
Bisected at https://invent.kde.org/utilities/konsole/-/merge_requests/528
Comment 2 Bug Janitor Service 2021-12-08 03:19:35 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/utilities/konsole/-/merge_requests/547
Comment 3 Fushan Wen 2021-12-08 07:07:50 UTC 
Git commit f3d43ae3cf02988b49c67ec6057e19931baaeedc by Fushan Wen.
Committed on 08/12/2021 at 07:03.
Pushed by fusionfuture into branch 'master'.

TerminalPainter: Pass nextPos to hasSameWidth

This avoids jumpy selections when the text contains both English and
non-English characters.

M  +1    -1    src/terminalDisplay/TerminalPainter.cpp

https://invent.kde.org/utilities/konsole/commit/f3d43ae3cf02988b49c67ec6057e19931baaeedc