| Summary: | Kicker does not return to normal width after deleting or clearing the text in the search field | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Patrick Silva <bugseforuns> |
| Component: | Application Menu (Kicker) widget | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | egor.kuznetsov.26, me, minelorderagon16, nate, notmart |
| Priority: | NOR | Keywords: | qt6 |
| Version First Reported In: | 6.1.3 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/09ac96239aa4fcbe48478d2b35e77db1baa2ba17 | Version Fixed/Implemented In: | 6.4.0 |
| Sentry Crash Report: | |||
| Attachments: | screen recording | ||
*** Bug 477768 has been marked as a duplicate of this bug. *** i find this happening only sometimes with no clear patterns to distinguish when it happens and when it doesn't (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. I can confirm that; thanks. *** Bug 501426 has been marked as a duplicate of this bug. *** A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/3027 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 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 |
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