Bug 374671 - Breeze Icons missing dark icons
Summary: Breeze Icons missing dark icons
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Bundle-MacOS (show other bugs)
Version: 5.4.0
Platform: macOS (DMG) macOS
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-07 01:15 UTC by Geoff King
Modified: 2017-01-21 11:32 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.5.0


Attachments
icons (29.28 KB, image/png)
2017-01-07 01:15 UTC, Geoff King
Details
icons2 (27.97 KB, image/png)
2017-01-07 01:16 UTC, Geoff King
Details
update screenshot (212.83 KB, image/png)
2017-01-15 21:28 UTC, Geoff King
Details

Note You need to log in before you can comment on or make changes to this bug.
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.