Bug 472007

Summary:	Navigating between characters of the search field's text sometimes replaces text with overlapping words you didn't type
Product:	[Plasma] krunner	Reporter:	Nate Graham <nate>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	alexander.lohnau, natalie_clarius
Priority:	NOR	Keywords:	qt6, regression
Version:	master
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/-/commit/3fdc9b1df6d6f09f2591198d2f05aab4c4172e26	Version Fixed In:	6.0
Sentry Crash Report:

Description Nate Graham 2023-07-06 16:22:13 UTC

STEPS TO REPRODUCE
1. Open KRunner
2. Type "104F"
3. Hit the left arrow key any number of times
4. Hit the right arrow key


OBSERVED RESULT
The text in the text field is replaced with the words "Search…" and "default" overlapping one another


EXPECTED RESULT
The insertion point moves between the characters without replacing the text you typed with anything


SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 38
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.240.0
Qt Version: 6.5.1
Kernel Version: 6.3.8-200.fc38.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 8 × Intel® Core™ i7-10510U CPU @ 1.80GHz
Memory: 15.2 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics


ADDITIONAL INFORMATION
If, when this happens, you hit the right arrow key a second time, the placeholder "Search…" text disappears and the text field's test is changes to just "default".

Very likely caused by https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2915, but I'm not able to bisect at the moment.

Comment 1 Bug Janitor Service 2023-07-06 16:32:41 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3047

Comment 2 Alexander Lohnau 2023-07-07 04:18:44 UTC

Git commit 3fdc9b1df6d6f09f2591198d2f05aab4c4172e26 by Alexander Lohnau, on behalf of Nate Graham.
Committed on 07/07/2023 at 04:17.
Pushed by alex into branch 'master'.

krunner: only replace with completion text on right arrow when relevant

We should only do this when the cursor position is at the end of the
typed text and there is faded completion text after it. At all other
times, the right arrow key should do normal navigation among characters,
or else pressing it can produce weird and unexpected results.
FIXED-IN: 6.0

M  +5    -1    krunner/qml/RunCommand.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/3fdc9b1df6d6f09f2591198d2f05aab4c4172e26