Bug 383615

Summary: "File type options" window has wrong icon under Wayland
Product: [Plasma] kde-cli-tools Reporter: Patrick Silva <bugseforuns>
Component: generalAssignee: Aleix Pol <aleixpol>
Status: RESOLVED FIXED    
Severity: normal CC: elvis.angelaccio, kde, meven29
Priority: NOR Keywords: wayland-only
Version First Reported In: 5.16.2   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: screenshot

Description Patrick Silva 2017-08-17 03:00:05 UTC
Created attachment 107323 [details]
screenshot

open dolphin, right click on any file/folder
select "properties", click "File type options" button... wayland generic icon (see the screenshot).
Click on any "Add" button... same thing.
Close "Add" window, click on second tab, click on "Add" button... wrong icon again.
Comment 1 Elvis Angelaccio 2017-08-17 08:24:09 UTC
This window comes from kde-cli-tools (keditfiletype5 binary).
Comment 2 Kai Uwe Broulik 2018-01-19 09:22:22 UTC
It announces a desktop file of "org.kde.keditfiletype" but there is no such file, instead it is "filetypes.desktop" and not in applications but in /usr/share/kservices5
Comment 3 Kai Uwe Broulik 2019-07-02 07:42:00 UTC
Probably needs a desktop file installed
Comment 4 Méven Car 2019-10-17 08:05:47 UTC
I can't reproduce this with Plasma 5.17.

Can you confirm Patrick so we can close it.
Comment 5 Patrick Silva 2019-10-17 12:10:57 UTC
Still reproducible on my system.

Operating System: Arch Linux 
KDE Plasma Version: 5.17.0
KDE Frameworks Version: 5.63.0
Qt Version: 5.13.1
Comment 6 Méven Car 2019-10-17 13:18:21 UTC
I didn't test properly before.
I do reproduce it locally.
Comment 7 Méven Car 2019-10-18 14:19:50 UTC
Git commit 9b2c7dec61de7425b62b1589d635eaa41ec77e97 by Méven Car.
Committed on 18/10/2019 at 14:19.
Pushed by meven into branch 'master'.

[KEditFileType] Add a desktop file

Summary:
Allow to display its Icon in Wayland

Reviewers: broulik, ngraham

Reviewed By: broulik

Subscribers: kossebau, plasma-devel

Tags: #plasma

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

M  +1    -0    keditfiletype/CMakeLists.txt
A  +8    -0    keditfiletype/org.kde.keditfiletype.desktop

https://commits.kde.org/kde-cli-tools/9b2c7dec61de7425b62b1589d635eaa41ec77e97