Bug 374671

Summary: Breeze Icons missing dark icons
Product: [Applications] digikam Reporter: Geoff King <gsking1>
Component: Bundle-MacOSAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 5.4.0   
Target Milestone: ---   
Platform: macOS (DMG)   
OS: macOS   
Latest Commit: Version Fixed In: 5.5.0
Sentry Crash Report:
Attachments: icons
icons2
update screenshot

Description Geoff King 2017-01-07 01:15:58 UTC
Created attachment 103246 [details]
icons

I am trying out the 5.4.0 beta on Mac OS X.  I want to use the "ShadeOfGray" or "GrayCard" theme.  I went to the Miscellaneous Settings and tried to change the Icon Theme, but there is no options for a "dark" icon set.  The only two options are "Breeze" and "Use Icon Theme From System". Without the dark icons, the icons are invisible with these gray themes. 

An example is shown in the attachment.
Comment 1 Geoff King 2017-01-07 01:16:31 UTC
Created attachment 103247 [details]
icons2
Comment 2 caulier.gilles 2017-01-07 09:13:41 UTC
I identified that only standard breeze icon path is bundled in rcc file, not breeze-dark. The code relevant is at this line :

https://cgit.kde.org/digikam-software-compilation.git/tree/project/bundles/macports/icon-rcc/CMakeLists.txt#n24

This is true for AppImage, MXE and Macports.

Gilles Caulier
Comment 3 caulier.gilles 2017-01-15 15:27:01 UTC
Git commit a1fea8affc57df5008407b7e2c494d3e0be2b6c4 by Gilles Caulier.
Committed on 15/01/2017 at 15:25.
Pushed by cgilles into branch 'master'.

manage breeze and breeze-dark icon themes from ressource files.
Related: bug 374313

M  +39   -8    libs/widgets/mainview/dxmlguiwindow.cpp

https://commits.kde.org/digikam/a1fea8affc57df5008407b7e2c494d3e0be2b6c4
Comment 4 caulier.gilles 2017-01-15 15:28:31 UTC
Git commit a5664d2515aef4cabbe5b85e79c9304d384886e4 by Gilles Caulier.
Committed on 15/01/2017 at 15:27.
Pushed by cgilles into branch 'master'.

package breeze and breeze-drask icons set as ressource files
Store files in AppImage bundles
Related: bug 374313

M  +1    -0    project/bundles/appimage/04-build-appimage.sh
M  +13   -5    project/bundles/appimage/icon-rcc/CMakeLists.txt

https://commits.kde.org/digikam-software-compilation/a5664d2515aef4cabbe5b85e79c9304d384886e4
Comment 5 caulier.gilles 2017-01-15 16:42:36 UTC
Git commit bfc94680177ebc360fa11fe3e67dc5fa713562e8 by Gilles Caulier.
Committed on 15/01/2017 at 16:41.
Pushed by cgilles into branch 'master'.

package breeze and breeze-drask icons set as ressource files
Store files in MacOS bundles
Related: bug 374313

M  +1    -0    project/bundles/macports/04-build-installer.sh
M  +13   -5    project/bundles/macports/icon-rcc/CMakeLists.txt
M  +1    -1    project/bundles/macports/installer/digikam.pkgproj

https://commits.kde.org/digikam-software-compilation/bfc94680177ebc360fa11fe3e67dc5fa713562e8
Comment 6 caulier.gilles 2017-01-15 18:10:22 UTC
Git commit 6a762e3d50f6cbb45c7db65ce5e0f814b155da6a by Gilles Caulier.
Committed on 15/01/2017 at 18:09.
Pushed by cgilles into branch 'master'.

package breeze and breeze-drask icons set as ressource files
Store files in MacOS bundles
Related: bug 374313

M  +1    -0    project/bundles/mxe/04-build-installer.sh
M  +14   -6    project/bundles/mxe/icon-rcc/CMakeLists.txt

https://commits.kde.org/digikam-software-compilation/6a762e3d50f6cbb45c7db65ce5e0f814b155da6a
Comment 7 caulier.gilles 2017-01-15 18:12:35 UTC
Geoff,

A new 5.5.0-pre release MacOS package is now available for testing at usual place :

https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM

Gilles Caulier
Comment 8 Geoff King 2017-01-15 21:28:42 UTC
Created attachment 103433 [details]
update screenshot
Comment 9 Geoff King 2017-01-15 21:31:21 UTC
Hello, 
I tried the 5.5.0-pre release MacOS package and the icons look great throughout the main interface. Thank You.  

However, One thing to note is that icons in the menus are now light in color compared to the text (but still visible).  Can these be made darker like other text in the menu?  See the screenshot.