Bug 480692

Summary: Whitelist Calculator runner so that calculations/conversions can be done using a search
Product: [Plasma] plasmashell Reporter: Henning <boredsquirrel>
Component: Application Menu (Kicker)Assignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: wishlist CC: dougshaw77, fanzhuyifan, nate
Priority: NOR    
Version: 5.92.0   
Target Milestone: 1.0   
Platform: unspecified   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=389146
Latest Commit: Version Fixed In: 6.1
Sentry Crash Report:

Description Henning 2024-02-01 17:11:48 UTC
I thought the Plasma Search modules would be all applied on every menu, it seems every menu has to integrate them seperately?

On default Kicker, 1+2 gives no output.

If this is an issue, it is reported now. It seems most plasma search modules are not integrated in kicker.

--- Software ---
OS: Fedora Linux 40.20240130.0 (Kinoite Prerelease)
KDE Plasma: 5.92.0
Kernel: 6.8.0-0.rc0.20240112git70d201a40823.5.fc40.x86_64
Compositor: wayland
Comment 1 Doug 2024-02-02 05:18:49 UTC
It is working for me.  The results are displayed next to the calculator icon instead of "Kcalc", so it's a little unfinished looking.

Operating System: KDE neon Testing Edition
KDE Plasma Version: 6.0.0
KDE Frameworks Version: 5.249.0
Qt Version: 6.6.1
Kernel Version: 6.5.0-15-generic (64-bit)
Graphics Platform: Wayland
Graphics Processor: AMD Radeon Pro WX 3200 Series
Comment 2 fanzhuyifan 2024-02-02 06:28:07 UTC
Could you provide a screenshot? I am pretty sure calculator is enabled by default.

Also if you go to settings - plasma search, you can check if calculator is enabled.
Comment 3 Henning 2024-02-11 15:36:57 UTC
are you using default kickoff, or the way less feature complete kicker? This was happening for me on kicker, where basically all plasma search modules are missing
Comment 4 Nate Graham 2024-02-15 05:22:38 UTC
This is intentional for Kicker at the moment due to Bug 389146. If and when that's fixed, we can consider whitelisting the calculator runner here (and potentially others too)
Comment 5 Henning 2024-02-15 13:30:31 UTC
thanks for the update!
Comment 6 Bug Janitor Service 2024-03-06 22:14:29 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2090
Comment 7 Nate Graham 2024-03-08 16:14:51 UTC
Git commit 9244d38295e2d02437aeba819d830770294e1378 by Nate Graham.
Committed on 08/03/2024 at 16:06.
Pushed by ngraham into branch 'master'.

applets/kicker: add power, session, calc and unit converter runners

There is a limitation on the number of runners we can add here since
every runner adds a column, making the popup wider and wider. However
these runners return results that are infrequently matched by accident,
limiting the frequency with which they will appear and clutter up the
UI.

As such, I think adding them does not result in a worse UI and we can
safely do it even before we fix it to not grow wider with every new
runner's search results.
Related: bug 375857
FIXED-IN: 6.1

M  +6    -5    applets/kicker/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/9244d38295e2d02437aeba819d830770294e1378