Skip to content
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

[BUG] App icon (default_icon) is not crisp #116

Closed
kryptonit1 opened this issue Aug 25, 2022 · 2 comments · Fixed by #134
Closed

[BUG] App icon (default_icon) is not crisp #116

kryptonit1 opened this issue Aug 25, 2022 · 2 comments · Fixed by #134
Assignees
Labels
bug Something isn't working
Milestone

Comments

@kryptonit1
Copy link

kryptonit1 commented Aug 25, 2022

Describe the bug
This is not really a bug, but I would not consider it a feature either. When you rebrand IBM Notifier with your own app icon it becomes clear that the app icon is not very crisp. I first thought that this was due to some error on my part, but it seems like it's not.

This is how my custom app icon looks like in the pop-up UI when an icon path has not been specified:
default_icon

This is how the app icon looks like when I create and specify a separate ICNS-file that I use in the pop-up UI:
separately_created_icns

I think the difference in crispness is due to the lack of different icon sizes for the default_icon for the pop-up UI in the Xcode project:
xcode_default_icon

This is not a problem when you push out banners or alerts since it seems like it might be utilising the AppIcon asset under the Notification Agent in the Xcode project which contains more icon sizes:
xcode_notification_icons

To Reproduce
Steps to reproduce the behavior:

  1. Rebrand IBM Notifier with a custom icon using the instructions in the wiki.
  2. Push out a pop-up notification and take a screenshot of the pop-up.
  3. Push out a pop-up notification with an icon_path specified. You can use any default macOS ICNS icon under /System/Library/CoreServices/CoreTypes.bundle/Contents/Resources. Take a screenshot of the pop-up.
  4. Compare the difference in crispness between the icons.

It's possible to work around this by specifying an icon path to a separately created ICNS, but it would be nice if the default_icon contained more sizes so that you would not have to create a separate ICNS in order for the icon to look more crisp.

Expected behavior
The app icon should look as crisp as the default macOS icons.

Screenshots
Please see above.

Desktop (please complete the following information):

  • OS: macOS 12.5.1
  • Project version: 2.8.0

Additional context
N/A

@kryptonit1 kryptonit1 added the bug Something isn't working label Aug 25, 2022
@SMartorelli
Copy link
Member

Hi @kryptonit1 I know what you mean and I also observed this. Tbh I didn't noticed the difference between custom path icons and the default one without your example. We'll work to improve this in the next iteration!

@SMartorelli SMartorelli self-assigned this Aug 25, 2022
@SMartorelli SMartorelli added this to the Version 2.9.0 milestone Aug 25, 2022
@kryptonit1
Copy link
Author

Thank you for your quick reply as always! :)

I think since the IBM Notifier default icon is black you don't see the difference as much as when you have a custom icon using other colors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants