Bug 390594

Summary:	"Application Addons" is erroneously full of "Plasma Addons"
Product:	[Applications] Discover	Reporter:	Nate Graham <nate>
Component:	discover	Assignee:	Aleix Pol <aleixpol>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	5.12.0
Target Milestone:	---
Platform:	Neon
OS:	Linux
Latest Commit:	https://commits.kde.org/discover/4959871c57d71209fff14eaea6b31b2c1d953d13	Version Fixed In:	Plasma 5.12.2

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.

https://www.reddit.com/r/kde/comments/7y0dvr/plasma_the_road_to_perfection_is_paved_with_bugs/dudc0fk/

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

Summary:
FIXED-IN: Plasma 5.12.2

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

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

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

https://commits.kde.org/discover/4959871c57d71209fff14eaea6b31b2c1d953d13