flameshot icon indicating copy to clipboard operation
flameshot copied to clipboard

Inconsistency of app icon across linux, windows and macos

Open cmpute opened this issue 3 weeks ago • 2 comments

Flameshot Version

Flameshot v13.3.0

Installation Type

Linux, MacOS, or Windows Package manager (apt, pacman, eopkg, choco, brew, ...)

Operating System type and version

MacOS

Description

On windows and linux, the app icon is colored with high-resolution:

Image (linux)

Image (windows)

On MacOS, the app icon is monochrome with low-resolution (which uses the same icon style as the tray icon).

Image

IMO, the app icon on MacOS should be the same with linux and windows.

Steps to reproduce

None, just install the app.

Screenshots or screen recordings

No response

System Information

Linux: PopOS 24.04 LTS beta Windows: Win11 24H2 Mac: MacOS Seqouia 15.6

cmpute avatar Dec 03 '25 04:12 cmpute

The Mac icon is intentionally monochrome to better fit the other items in the Mac toolbar.

You can compile the Mac release with the color icon if you want to, but the default release from us will be monochrome

borgmanJeremy avatar Dec 03 '25 11:12 borgmanJeremy

@borgmanJeremy I'm not sure which 'toolbar' you meant. When I search 'Mac toolbar', I only got the toolbar of Mac Finder (https://support.apple.com/guide/mac-help/mchlp3011/mac), which seems to be irrelevant here. If you meant the Mac 'menubar' (https://support.apple.com/guide/mac-help/mchl4af84660/mac), then it's not related to my changes. The icon used for the menubar is defined here:

https://github.com/flameshot-org/flameshot/blob/7ed3cfc83eda4bd33f5044041075689bb517a314/src/utils/globalvalues.cpp#L44-L51

which, as you can see, uses the flameshot.mask.png instead of the flameshot.monochrome.png file changed in my PR (for newer MacOS version). In another word, change the icon path used for app icon packing will not affect the icon used in the menubar. (This is regardless of which MacOS version the user is using, because the path for menubar and for the app icon is two separate string literal constant)

cmpute avatar Dec 03 '25 12:12 cmpute