Bug 381161

Summary: Discover uses a low resolution icon for Kdenlive
Product: [KDE Neon] neon Reporter: KDE Neon user <emailmeat>
Component: generalAssignee: Neon Bugs <neon-bugs>
Status: REOPENED ---    
Severity: normal CC: aleixpol, fonsecaregis33, jr, macanon.rodriguez, neon-bugs, sitter
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: screenshot
The actual file that kdenlive uses
What it looks like on ArchLinux

Description KDE Neon user 2017-06-13 15:36:07 UTC
Created attachment 106080 [details]
screenshot

See screenshot.
Comment 1 macanon.rodriguez 2017-06-14 04:45:14 UTC
I have the same issue.
Comment 2 Aleix Pol 2017-06-15 12:40:59 UTC
This is not the image that is being installed by kdenlive, I'm unsure where it's coming from. I'm assuming it's a bug in Neon packaging.

Neoners, can you please take a look?
Comment 3 Aleix Pol 2017-06-15 12:41:49 UTC
Created attachment 106110 [details]
The actual file that kdenlive uses
Comment 4 Aleix Pol 2017-06-15 12:42:48 UTC
Created attachment 106111 [details]
What it looks like on ArchLinux
Comment 5 Harald Sitter 2017-06-16 07:16:36 UTC
Pretty sure that is because our appstream data has org.kde.kdenlive while it should have a .desktop suffix (which it does in the appstream data and ubuntu) so what is listed is in fact the ubuntu one along with the ubuntu icon.

$ appstreamcli search kdenlive
Identifier: org.kde.kdenlive [desktop-application]
Name: Kdenlive
Summary: Nonlinear video editor by KDE
Package: kdenlive
---
Identifier: org.kde.kdenlive.desktop [desktop-application]
Name: Kdenlive
Summary: Nonlinear video editor for KDE
Package: kdenlive

Why that is I can't say, the entire appstream stack we use is yet again outdated.
Comment 6 Jonathan Riddell 2017-06-16 10:48:16 UTC
hard to update appstream-generator, latest git version requires newer meson and something called gir-to-d which we don't have
Comment 7 Harald Sitter 2017-06-16 13:53:34 UTC
new meson is in xenial-backports, gir-to-d simply needs building. beyond that what is needed is a new libappstream, in fact out of all this the new libappstream is the bit that impacts things outside the asgen build environment.

also FTR ldc (llvm based d compiler) isn't playing ball, so gcc probably would have to be used instead.
Comment 8 Aleix Pol 2017-06-19 16:07:44 UTC
*** Bug 381030 has been marked as a duplicate of this bug. ***
Comment 9 Jonathan Riddell 2017-07-10 15:41:26 UTC
appstream-generator updated

I now see two entries one with a nice icon and one with no icon, most strange

appstreamcli returns two identifiers
Identifier: org.kde.kdenlive 
Identifier: org.kde.kdenlive.desktop
Comment 10 Harald Sitter 2017-07-10 18:23:07 UTC
With new generator? becuase the raw data also had two entries when I looked it. hence why I wanted a new stack, can't really file a bug report with upstream if we don't know it's affecting master
Comment 11 Ralf Habacker 2018-02-10 12:36:58 UTC
Git commit c1d3b78bea1ece370dfbefea1723d051c876c8d4 by Ralf Habacker.
Committed on 10/02/2018 at 12:35.
Pushed by habacker into branch '4.8'.

Fix vertical diagram range in case there is only one value displayed

The previous applied fix introduced bug 389756.
Related: bug 389756
FIXED-IN:4.8.2

M  +13   -4    kmymoney/reports/kreportchartview.cpp

https://commits.kde.org/kmymoney/c1d3b78bea1ece370dfbefea1723d051c876c8d4
Comment 12 Aleix Pol 2018-02-12 00:25:04 UTC
I'm pretty sure this fix was unrelated.
Comment 13 Jonathan Riddell 2018-02-12 10:52:29 UTC
This works fine in KDE neon User Editions
https://i.imgur.com/4yAw1Lh.png
it's in dev editions where we don't have appstream metadata that it looks crap