-
Notifications
You must be signed in to change notification settings - Fork 2
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
Tray icon size #103
Comments
Does it look like it is supposed to look on Windows? |
Thanks for the screenshots. OK so it is also too big for you on Mac, while on Windows it looks a little small compared to other icons (the pipes could stick out a bit from the recommended area). I'll create a PR for the Mac part. |
See #111. |
tested and merged. So this is now just a Windows issue. |
For the windows icon, the problem is that the icon rendering size is fixed by windows. The issue with the icon here is that it has a large part of it transparent, especially on borders due to the "pipes" in the icon To have a visually more visible icon, we might need to make a more "cropped" version of it (with shorter pipes for example). |
On macOS, the tray icon is two times too large on retina displays.
Some time ago I made a patch that addressed this problem. I need to have a look at it and see if I can make a PR from it. That patch also includes a nicer designed tray icon. It is derived from the known Pipeline icon, but slightly more simplistic. It is also monochromatic like tray icon ought to be on Mac, and the black and white is inverted when clicked. Note that this doesn't change anything for Windows.
The text was updated successfully, but these errors were encountered: