Differential D26849

Reorder Relevance of runner result
AcceptedPublic
Actions

Authored by meven on Jan 22 2020, 7:05 PM.

Details

Reviewers
ngraham
Group Reviewers
Plasma
Summary

Review the relevance so that when we have a match for an application we have:

  • Windows of this app
  • Application
  • command

Instead of currently :

  • Command
  • Application
  • Windows of this app

CCBUG: 414804

Test Plan

Before:

After:

Diff Detail

Repository
R120 Plasma Workspace
Branch
runner-relevance
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 21526
Build 21544: arc lint + arc unit
meven created this revision.Jan 22 2020, 7:05 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 22 2020, 7:05 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
meven requested review of this revision.Jan 22 2020, 7:05 PM
meven edited the test plan for this revision. (Show Details)Jan 22 2020, 7:07 PM
ngraham added a comment.Jan 22 2020, 8:15 PM

This makes perfect sense to me.

See also https://bugs.kde.org/show_bug.cgi?id=359627.

ngraham accepted this revision.Jan 22 2020, 8:15 PM
This revision is now accepted and ready to land.Jan 22 2020, 8:15 PM

Revision Contents
Changeset List

Diff 74163

View Options

runners/shell/shellrunner.cpp

Loading...
View Options

runners/windows/windowsrunner.cpp

Loading...