| Summary: | First item in first column of Kicker search results is hidden by search field | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Martin Bednar <martin+kde> |
| Component: | Application Menu (Kicker) | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | gabriele.girdv, nate |
| Priority: | HI | Keywords: | qt6 |
| Version: | 6.0.1 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/f9f87a3c89c6082e8171c4e87df01fe8df8280c9 | Version Fixed In: | 6.0.2 |
| Attachments: |
showing file result hidden
showcasing bookmarks showcasing commandline |
||
Created attachment 166608 [details]
showcasing bookmarks
Created attachment 166609 [details]
showcasing commandline
Comment on attachment 166609 [details]
showcasing commandline
This file search has 3 results, command line 1 result.
Can reproduce. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2099 Git commit e0582b167be904abedb2332ad3f459eab52e1d4e by Nate Graham, on behalf of Marco Martin. Committed on 08/03/2024 at 17:23. Pushed by ngraham into branch 'master'. Rework logic to know who is the first visible results view We have a horizontal layout of runner results that separate the results by category. In order to show enough space for the text field and to disable the first separator line, it assumed the item with index===0 was the first element. However if there aren't results which are applications, this assumption would break and the layout was wrong, so check that this column is actually the first visible child. FIXED-IN: 6.0.2 M +1 -1 applets/kicker/package/contents/ui/MenuRepresentation.qml M +8 -7 applets/kicker/package/contents/ui/RunnerResultsList.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/e0582b167be904abedb2332ad3f459eab52e1d4e Git commit f9f87a3c89c6082e8171c4e87df01fe8df8280c9 by Nate Graham, on behalf of Marco Martin. Committed on 08/03/2024 at 17:25. Pushed by ngraham into branch 'Plasma/6.0'. Rework logic to know who is the first visible results view We have a horizontal layout of runner results that separate the results by category. In order to show enough space for the text field and to disable the first separator line, it assumed the item with index===0 was the first element. However if there aren't results which are applications, this assumption would break and the layout was wrong, so check that this column is actually the first visible child. FIXED-IN: 6.0.2 (cherry picked from commit e0582b167be904abedb2332ad3f459eab52e1d4e) f5237fb8 Rework logic to know who is the first visible results view db6d7462 fix resizing M +1 -1 applets/kicker/package/contents/ui/MenuRepresentation.qml M +8 -7 applets/kicker/package/contents/ui/RunnerResultsList.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/f9f87a3c89c6082e8171c4e87df01fe8df8280c9 *** Bug 482771 has been marked as a duplicate of this bug. *** Indeed, this is a lot better. Thanks! |
Created attachment 166607 [details] showing file result hidden When a kicker search produces no applications results, the applications pane is hidden. In this case, all of the "bookmarks" "files" and "command line" panes have their lowermost result hidden by the search bar. This result is merely hidden, accessible if the pane is longer than the search bar. STEPS TO REPRODUCE 1. Open kicker 2. search for something with no applications result 3. navigate with keyboard to select the hidden result EXPECTED RESULT: bookmarks/file results should behave the same as applications results: not be covered by the search bar KDE Plasma Version: 6.0.1 KDE Frameworks Version: 6.0.0 Qt Version: 6.6.2