Bug 477005 - Kicker does not return to normal width after deleting or clearing the text in the search field
Summary: Kicker does not return to normal width after deleting or clearing the text in...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Application Menu (Kicker) widget (other bugs)
Version First Reported In: 6.1.3
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: qt6
: 477768 501426 (view as bug list)
Depends on:
Blocks:
 
Reported: 2023-11-14 18:17 UTC by Patrick Silva
Modified: 2025-05-29 07:51 UTC (History)
5 users (show)

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


Attachments
screen recording (1.58 MB, video/webm)
2023-11-14 18:17 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2023-11-14 18:17:01 UTC
Created attachment 163159 [details]
screen recording

STEPS TO REPRODUCE
1. open Kicker
2. search for "kde" - search results are shown in two columns
3. clear the search field

OBSERVED RESULT
Kicker does not return to normal width. Please watch the attached screen recording.

EXPECTED RESULT
Kicker returns to normal width after cleaning the search field

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.81.0
KDE Frameworks Version: 5.245.0
Qt Version: 6.6.0
Kernel Version: 6.5.7-060507-generic (64-bit)
Graphics Platform: Wayland
Comment 1 Nate Graham 2023-11-30 16:36:41 UTC
*** Bug 477768 has been marked as a duplicate of this bug. ***
Comment 2 Marco Martin 2025-01-29 13:05:00 UTC
i find this happening only sometimes with no clear patterns to distinguish when it happens and when it doesn't
Comment 3 Antonino Scordino 2025-02-06 21:21:18 UTC
(In reply to Marco Martin from comment #2)
> i find this happening only sometimes with no clear patterns to distinguish
> when it happens and when it doesn't

Seems like there is indeed a specific pattern behind this. When "forcefully" closing the start menu with ESC or its own button, width accordingly restores.

On the other hand, width change only persists when deleting the content of the search box manually, like the video demonstrates. That's the only case where I could trigger this bug manually.
Comment 4 Nate Graham 2025-02-08 01:00:10 UTC
I can confirm that; thanks.
Comment 5 Nate Graham 2025-03-13 20:06:53 UTC
*** Bug 501426 has been marked as a duplicate of this bug. ***
Comment 6 Bug Janitor Service 2025-05-27 22:19:43 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/3027
Comment 7 cwo 2025-05-28 16:39:21 UTC
Git commit 50c0e0e0ff280b299a56228e66a4d2550c04d778 by Christoph Wolk.
Committed on 28/05/2025 at 16:19.
Pushed by cwo into branch 'master'.

applets/kicker: fix and simplify representation size

The horizontal sizing of the full representation is rather complex and
due to circular dependencies somewhat buggy; in particular, searching
for something that yields multiple results columns then clearing the
search field will leave the menu column with the combined width of all
the columns, rather than shrinking to its normal size.

We simplify this by having the optimal widths bubble up from the
individual parts to the whole, rather than being circular in both
directions. This also allows us to re-enable dynamic sizing for the
(non-runner) full representation without the previous bugginess. (We
keep the maximum width somewhat more limited at 1.5 times the normal
width, as the menu is harder to operate when it is very wide, compared
to the submenus or runner columns). Along the way, we use optional
chaining to get rid of a runtime error that occasionally happens when
closing Kicker.
FIXED-IN: 6.4.0

M  +1    -1    applets/kicker/package/contents/ui/ItemListView.qml
M  +6    -5    applets/kicker/package/contents/ui/MenuRepresentation.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/50c0e0e0ff280b299a56228e66a4d2550c04d778
Comment 8 cwo 2025-05-29 07:51:04 UTC
Git commit 09ac96239aa4fcbe48478d2b35e77db1baa2ba17 by Christoph Wolk.
Committed on 29/05/2025 at 06:49.
Pushed by cwo into branch 'Plasma/6.4'.

applets/kicker: fix and simplify representation size

The horizontal sizing of the full representation is rather complex and
due to circular dependencies somewhat buggy; in particular, searching
for something that yields multiple results columns then clearing the
search field will leave the menu column with the combined width of all
the columns, rather than shrinking to its normal size.

We simplify this by having the optimal widths bubble up from the
individual parts to the whole, rather than being circular in both
directions. This also allows us to re-enable dynamic sizing for the
(non-runner) full representation without the previous bugginess. (We
keep the maximum width somewhat more limited at 1.5 times the normal
width, as the menu is harder to operate when it is very wide, compared
to the submenus or runner columns). Along the way, we use optional
chaining to get rid of a runtime error that occasionally happens when
closing Kicker.
FIXED-IN: 6.4.0


(cherry picked from commit 50c0e0e0ff280b299a56228e66a4d2550c04d778)

Co-authored-by: Christoph Wolk <cwo.kde@posteo.net>

M  +1    -1    applets/kicker/package/contents/ui/ItemListView.qml
M  +6    -5    applets/kicker/package/contents/ui/MenuRepresentation.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/09ac96239aa4fcbe48478d2b35e77db1baa2ba17