Perform KNS searches from the Featured page again
ClosedPublic

Authored by ngraham on Thu, Nov 7, 3:03 PM.

Details

Summary

We previously disabled this in https://bugs.kde.org/show_bug.cgi?id=399503 to improve
the relevancy of the search results. However since then, searching has been improved in
various other ways. For example, exact title matches are now shown first, and apps are
shown before KNS entries. Therefore, it seems reasonable to re-enable KNS searches from
the Featured page.

BUG: 413880
FIXED-IN: 5.18.0

Test Plan

Can now find "Fokus" from a search inittated while viewing the Featured page.

Diff Detail

Repository
R134 Discover Software Store
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Thu, Nov 7, 3:03 PM
Restricted Application added a project: Plasma. · View Herald TranscriptThu, Nov 7, 3:03 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Thu, Nov 7, 3:03 PM
apol added a comment.Thu, Nov 7, 7:00 PM

How about we target master for this?

libdiscover/backends/KNSBackend/KNSBackend.cpp
507

The comment should be //Accept global searches, the featured page queries by URL.

ngraham marked an inline comment as done.Thu, Nov 7, 7:06 PM

Sure, master only is fine.

ngraham edited the summary of this revision. (Show Details)Thu, Nov 7, 7:06 PM
ngraham updated this revision to Diff 69407.Thu, Nov 7, 7:06 PM
ngraham edited the summary of this revision. (Show Details)

Update comment

ngraham edited the summary of this revision. (Show Details)Thu, Nov 7, 7:07 PM
apol accepted this revision.Fri, Nov 8, 5:49 PM
This revision is now accepted and ready to land.Fri, Nov 8, 5:49 PM
This revision was automatically updated to reflect the committed changes.