Cleanup RunnerSyntaxes
Open, Needs TriagePublic

Description

The syntaxes were pretty much unused in KF5 and because the usage help for KRunner is in progress(https://phabricator.kde.org/T13073) we stumbled upon some issues.

Using the syntaxes for the single runner mode is hacky: We have multiple syntaxes in the runner and one default one. From this default one we grab the first example query and use it as a template in the UI. (See RunnerManager line 866) . But in the current implementation of the frontends we don't even allow empty queries.

So maybe we should just use the syntaxes for the usage help only, deprecate the unused methods and consequently remove this functionality in the RunnerManager in KF6.

Related Objects

StatusAssignedTask
Openalex
Openalex
alex created this task.Sep 14 2020, 3:34 PM
alex added a comment.EditedOct 7 2020, 4:25 PM

https://invent.kde.org/frameworks/krunner/-/merge_requests/32
https://invent.kde.org/frameworks/krunner/-/merge_requests/36

If the second MR has landed I would mark this task as "Waiting for KF6 to be branched". From that point on the only thing left to do if port the very few runners that use the methods that are about to be deprecated and remove the methods once KF6 gets branched.

alex moved this task from Backlog to Waiting on KF6 Branching on the KF6 board.Jan 9 2021, 8:58 AM
alex moved this task from Waiting on KF6 Branching to Done on the KF6 board.Feb 7 2023, 8:17 AM

All cleaned up in master now :)