Bug 390594 - "Application Addons" is erroneously full of "Plasma Addons"
Summary: "Application Addons" is erroneously full of "Plasma Addons"
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: 5.12.0
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Aleix Pol
Depends on:
Reported: 2018-02-17 05:29 UTC by Nate Graham
Modified: 2018-02-17 17:03 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: Plasma 5.12.2


Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2018-02-17 05:29:27 UTC
The distinction between "Application Addons" and "Plasma Addons" seems arbitrary and confusing. For example, "Application Addons" contains:
- color schemes
- comics
- emoticons
- kate snippets
- plymouth
- sddmtheme

With the exception of "Kate Snippets", all of these are systemwide in nature and I would expect them to be in "Plasma Addons" instead. And if there's only one item in that's truly appropriate for "Application Addons", maybe we should just have a single "Addons" category that includes everything.

This would simplify things and make searching for addons more reliable, as this was brought to my attention came up when a user tried to search "Plasma Addons" for cursor themes, quite understandably not knowing that they would be listed under "Application Addons" The search came up empty, so he concluded that we suck at searching.

Comment 1 Nate Graham 2018-02-17 15:43:02 UTC
Submitted a patch for this: https://phabricator.kde.org/D10606
Comment 2 Nate Graham 2018-02-17 16:58:06 UTC
Git commit 4959871c57d71209fff14eaea6b31b2c1d953d13 by Nathaniel Graham.
Committed on 17/02/2018 at 16:46.
Pushed by ngraham into branch 'Plasma/5.12'.

Put all systemwide add-ons in Plasma Addons category

FIXED-IN: Plasma 5.12.2

Test Plan:
Application Addons category now only has actual app-specific addons:

Plasma Addons category is now nice and full of actual Plasma (or other system-level) add-ons:

Reviewers: #discover_software_store, apol

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D10611

M  +4    -1    libdiscover/backends/KNSBackend/KNSBackend.cpp