| Summary: | Search results within a category/runner don't seem to be ranked by frequency anymore | ||
|---|---|---|---|
| Product: | [Plasma] krunner | Reporter: | insuna |
| Component: | general | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | alexander.lohnau, natalie_clarius, nate |
| Priority: | NOR | Keywords: | regression |
| Version First Reported In: | 6.3.1 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/frameworks/krunner/-/commit/413ad73fdab421e619a73f2f89c07825eb3e87c4 | Version Fixed/Implemented In: | 6.17 |
| Sentry Crash Report: | |||
| Attachments: | screenshot of plasma search settings | ||
|
Description
insuna
2025-03-02 19:14:03 UTC
This is just the standard KRunner-provided search; moving to KRunner. Can you attach a screen recording that shows the issue happening? And then can you also attach a screenshot showing the state of System Settings > Search > Plasma Search? Created attachment 179170 [details] screenshot of plasma search settings (In reply to Nate Graham from comment #1) > This is just the standard KRunner-provided search; moving to KRunner. > > Can you attach a screen recording that shows the issue happening? > > And then can you also attach a screenshot showing the state of System > Settings > Search > Plasma Search? Hello Nate, thank you for your reply. here's a video of me trying to start Discord from the search bar: https://youtu.be/UMAdwr6c5xM I hope the attachment thing works for the screenshot. Please do note that this is a feature request and not a bug report. I'm sure this is behaving as intended, I just think the intention should be configurable to make the search results more suited to my needs. Thanks. The reason I ask because KRunner is already supposed to order results within a category (as is happening here) by frequency of use. It seems like maybe it's not working though, as I can reproduce the same issue with those same two apps. If I type "disco" and open Discord 20 times using KRunner, Discover still always shows up first. Yeah, I can see in my ~/.local/share/krunnerstaterc file that under [PlasmaRunnerManager][History], the sole item in there gets updated, but under [PlasmaRunnerManager], LaunchCounts= does NOT get updated. So the ordering of items determined by launch counts in the past are now frozen. A possibly relevant merge request was started @ https://invent.kde.org/frameworks/krunner/-/merge_requests/197 Git commit 413ad73fdab421e619a73f2f89c07825eb3e87c4 by Nate Graham. Committed on 22/07/2025 at 16:05. Pushed by ngraham into branch 'master'. Save LaunchCounts immediately on launch if there were any matches Writing on every run *is* necessary, or else `LaunchCounts` won't get updated in a timely matter or at all. Replace that with simpler logic to save to the config file only if there were any launches. This also makes the autotest match the actual behavior. Partially reverts 7cb4ff7e812e670c81a60c2b3f3126290ee74c2e FIXED-IN: 6.17 M +0 -5 autotests/runnermanagerhistorytest.cpp M +0 -5 autotests/runnermanagertest.cpp M +1 -2 src/runnercontext.cpp https://invent.kde.org/frameworks/krunner/-/commit/413ad73fdab421e619a73f2f89c07825eb3e87c4 When can it be expected for this fix to be rolled out? It says fixed in 6.17, but I still have the exact same issue on krunner 6.19.0.. Right now there aren't even any launch counts included anymore: ``` [PlasmaRunnerManager][History] default=sett,disc,stra,fire ``` this is my entire krunnerstaterc file... |