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

Status appIcon no longer shown #40

Open
RobinDaugherty opened this issue Mar 9, 2021 · 2 comments
Open

Status appIcon no longer shown #40

RobinDaugherty opened this issue Mar 9, 2021 · 2 comments

Comments

@RobinDaugherty
Copy link

RobinDaugherty commented Mar 9, 2021

Not sure when, but several macOS versions ago, the app icon is forced to be the actual app icon. Only contentImage has any effect now.

But as of df82582 the appIcon and contentImage options were switched, making all guard notifications look the same:

["/opt/rbenv/versions/2.7.2/bin/terminal-notifier",
 "-title",
 "RSpec results",
 "-priority",
 "-2",
 "-message",
 "2 examples, 0 failures in 0.0185 seconds",
 "-contentImage",
 "/opt/rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/terminal-notifier-guard-1.7.0/icons/Guard.icns",
 "-appIcon",
 "/opt/rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/terminal-notifier-guard-1.7.0/icons/Success.icns"]

Screen Shot 2021-03-09 at 10 05 44

["/opt/rbenv/versions/2.7.2/bin/terminal-notifier",
 "-title",
 "RSpec results",
 "-priority",
 "-2",
 "-message",
 "2 examples, 2 failures in 1.1234 seconds",
 "-contentImage",
 "/opt/rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/terminal-notifier-guard-1.7.0/icons/Guard.icns",
 "-appIcon",
 "/opt/rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/terminal-notifier-guard-1.7.0/icons/Failed.icns"]

Screen Shot 2021-03-09 at 10 07 35

This is obviously unfortunate. I think it's been broken for at least a year.

I think the solution is to just swap appIcon and contentImage back to the way they were, along with a major version bump, so people sticking with older macOS versions can just lock to the old version.

An alternative is to include unicode characters in the message content/subtitle to indicate success/failure.

@gorodulin
Copy link

Please review&merge @lanej0's PR 🙏🏻

@dsnovaes
Copy link

dsnovaes commented Oct 27, 2022

Any solution to this yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants