Bug 435866

Summary: No ledger icon in the pane of the left side
Product: [Applications] kmymoney Reporter: Ralf Habacker <ralf.habacker>
Component: packagingAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal CC: andre+joost, me
Priority: NOR    
Version: 5.1.1   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: https://invent.kde.org/office/kmymoney/-/commit/32c7f2a16ad14f50834f43d9e338b939c05dcef1 Version Fixed In: 5.1.3
Bug Depends on:    
Bug Blocks: 435581    
Attachments: screenshot showing colored icons build from master branch

Description Ralf Habacker 2021-04-18 07:19:13 UTC 
SUMMARY

kmymoney has 20 png icons of its own under /share/kmymoney/icons, e.g. the ledger icon. Those were present in the latest stable windows build 5.06 and unstable 5.07. In 5.1.1 there is just no icon now.

STEPS TO REPRODUCE
1. Download portable package for kmymoney from https://kmymoney.org/snapshots.php
2. unpack 7z file
3. start kmymoney
4. inspect the icon pane of the left side

OBSERVED RESULT
ledger icon is not shown

EXPECTED RESULT
ledger icon should be displayed

SOFTWARE/OS VERSIONS
Windows: 10
KDE Frameworks Version: 5.65.0
Qt Version: 5.9
Comment 1 AndreJ 2021-04-18 11:00:53 UTC 
In Version 5.07, the icon was taken from
C:\Program Files\kmymoney5\share\kmymoney\icons\hicolor
while the rest of the icons was taken from the icontheme.rcc file which was build from the breeze iconset. I guess the ledger icon (and about 20 others from the same hicolor folder) have to be taken into the icontheme.rcc file.
Comment 2 Ralf Habacker 2021-04-24 12:38:23 UTC 
Created attachment 137866 [details]
screenshot showing colored icons build from master branch

The issue seems to be related to the 5.1 branch only, because it has been fixed in the master branch.
Comment 3 Dawid Wróbel 2021-06-30 06:08:50 UTC 
Git commit f700cfb15809ec7b5e182d9e3947a5c3250ac029 by Dawid Wróbel.
Committed on 30/06/2021 at 06:08.
Pushed by wrobelda into branch '5.1'.

Also fix bank/institution, account icons

M  +3    -2    kmymoney/icons/icons.cpp

https://invent.kde.org/office/kmymoney/commit/f700cfb15809ec7b5e182d9e3947a5c3250ac029
Comment 4 Dawid Wróbel 2021-06-30 06:31:48 UTC 
Git commit 483b954791ccf4d7cfdcd2da659918dc0f1059a0 by Dawid Wróbel.
Committed on 30/06/2021 at 06:31.
Pushed by wrobelda into branch '5.1'.

Also fix transaction icon

M  +4    -1    kmymoney/icons/icons.cpp

https://invent.kde.org/office/kmymoney/commit/483b954791ccf4d7cfdcd2da659918dc0f1059a0