Bug 396833

Summary: "Missing Backends" section sometimes shows actually installed backends
Product: [Applications] Discover Reporter: john <john.pelicans>
Component: discoverAssignee: Aleix Pol <aleixpol>
Status: RESOLVED FIXED    
Severity: normal CC: andrew.crouthamel, bugseforuns, nate
Priority: NOR    
Version: 5.13.3   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: screenshot of issue

Description john 2018-07-25 02:03:24 UTC
Created attachment 114104 [details]
screenshot of issue

There are a few issues on the settings page of Discover.

#1 Each subheading (Flatpak, OpenSUSE Tumbleweed (default), and Missing Backends) has too little left margin

#2 "Missing Backends" should be renamed to "Backends", especially when said backends are insalled
Comment 1 Nate Graham 2018-07-25 21:52:31 UTC
One issue per bug report please. See https://community.kde.org/Get_Involved/Bug_Reporting#One_issue_per_bug_report

I'll submit a patch to fix #1, so let's use this to track #2. This is the second report I've seen of this issue, so I think it's real.
Comment 2 Nate Graham 2018-07-25 23:35:20 UTC
Git commit 3cd95b4a4f4d80826d50dbdd2cc4b2ef783cf54e by Nathaniel Graham.
Committed on 25/07/2018 at 23:15.
Pushed by ngraham into branch 'master'.

Increase backend header text's left margin

Summary:
Give the backend name a bit more room so it doesn't look so cramped on the left side. Even though this makes the left edge of the text no longer align with the checkboxes, somehow it looks better overall to me.

Test Plan:
Before:
{F6139953}

After:
{F6139955}

Reviewers: apol, #discover_software_store, #vdg

Reviewed By: apol, #discover_software_store

Subscribers: plasma-devel

Tags: #plasma

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

M  +1    -0    discover/qml/SourcesPage.qml

https://commits.kde.org/discover/3cd95b4a4f4d80826d50dbdd2cc4b2ef783cf54e
Comment 3 john 2018-07-27 00:02:39 UTC
(In reply to Nate Graham from comment #1)
> One issue per bug report please. See
> https://community.kde.org/Get_Involved/Bug_Reporting#One_issue_per_bug_report
> 
> I'll submit a patch to fix #1, so let's use this to track #2. This is the
> second report I've seen of this issue, so I think it's real.

Sorry about that. I added the second one without thinking :)

Thanks for the fast patch!
Comment 4 Andrew Crouthamel 2018-09-22 03:01:55 UTC
I can confirm this on Manjaro. I installed the flatpak and packagekit backends, and they both list under "Missing Backends".

KDE Plasma Version: 5.13.5
KDE Frameworks Version: 5.50.0
Qt Version: 5.11.1
Comment 5 Aleix Pol 2018-09-24 16:12:38 UTC
Git commit 36e90a76d984e813448c75ad3f4fb9d0b14be002 by Aleix Pol.
Committed on 24/09/2018 at 16:11.
Pushed by apol into branch 'Plasma/5.14'.

Make sure we only show uninstalled backends

M  +1    -0    discover/qml/SourcesPage.qml

https://commits.kde.org/discover/36e90a76d984e813448c75ad3f4fb9d0b14be002