-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MacOS White Trayicon #2017
Comments
I think almost most applications in MacOS use monochrome icons. I haven't worked with MacOS for some years now, but that's what I remember. Considering that we already have monochrome icons, it in theory should be easy to either change it or, similar to what bartender app does, let the user to decide which tray icon they want to use: |
@mmahmoudian we actually dont directly have a monochrome icon. Other people have themed the icon here for example. Then on linux Qt picks up that themed icon first. On Mac the icon is actually compiled into the binary we would need to copy the svg from that git repo into our source tree and build the Mac version against it. If we do that we need to modify the svg to include its license (GPLv3) and point back to the original author. |
@borgmanJeremy interesting! I didn't know this. But about adding the SVG, we have the actual Flameshot icon. I think we can put together a monochrome stroke only version. This way it would be our own svg and we can license it to our project's preference. |
Yeah I'm not worried about the licensing, just want to make sure we credit properly. But yeah by all means if someone wants to make a new one that's fine with me. |
Hi @mmahmoudian What ending has a monochrome icon? best regards |
@Faetu If by ending you mean file extension or file format, would be nice to have SVG file. You can use the content of the following folder: Are you planning to make monochrome icon for Flameshot? |
Hi @mmahmoudian Yes! Because it is easy for me. I done this for my linux tray icons. It will be done by tonight. best regards |
Hi @mmahmoudian Its done! Now, how to continue to make this public and downloadable for everyone? best regards |
@borgmanJeremy @veracioux what do you think is the best approach? Would it be good if @Faetu create a PR with these new icons in the same folder as the other icons: https://github.com/flameshot-org/flameshot/tree/master/data/img/app |
I think it's better that one of the developers creates the PR, because the code needs to be changed as well. Of course ensuring proper credit to @Faetu. I could do it, but I don't have a Mac VM set up (yet), so I think it's better that someone else do it. |
Ok cool I'll do it. |
I think maybe we should take this opportunity to add a config option to switch between color and monochrome icons? Im certain someone on windows will ask for the same thing. The alternative to is to simply package the monochrome icon on Mac when we build the .dmg. |
@borgmanJeremy We are adding too many config options 😄. |
I think the issue isn't the number of options, but how they are organized in the config screen. |
As long as customisability doesn't affect the performance I personally don't see any issues in that. In contrary I think it would make it much easier for people to adopt it to their workflow. |
Okay so adding this as a run time configuration turned out to be pretty complicated to support across all the different DE's and operating systems. Not impossible, but as a first step I made it a build time option. Feel free to test in on #2071 but it seems to work for me |
Can you please open a separate issue for that? Icons in Linux are managed totally different and I don't want to clog this issue with that discussion. As a quick answer if you google XFCE change icon theme you will find it. But feel free to open something on the GitHub discussions if you need help. |
My bad. I've created issue #2083. |
Merged new icon on Macos |
Bug with multiple monitors setup, icon flattened on non active screen |
Is your feature request related to a problem?
Can we have MacOS White Tray Icon?
its not good enough for my eye 😄
any simple way to make this possible?
The text was updated successfully, but these errors were encountered: