Bug 480692 - Whitelist Calculator runner so that calculations/conversions can be done using a search
Summary: Whitelist Calculator runner so that calculations/conversions can be done usin...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Application Menu (Kicker) (show other bugs)
Version: 5.92.0
Platform: unspecified Linux
: NOR wishlist
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-01 17:11 UTC by Henning
Modified: 2024-03-08 16:14 UTC (History)
3 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/plasma-desktop/-/commit/9244d38295e2d02437aeba819d830770294e1378
Version Fixed In: 6.1

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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