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

Notification Icon #6

Closed
iakshay opened this Issue Jul 30, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@iakshay

iakshay commented Jul 30, 2013

Is it possible to change the notification icon? I'm getting Finder icon right now.

Also would be nice to have a command line argument for volume and alarm sound. So it can be easily customised by the users from the workflow page.

Thanks for the awesome workflow!

@dbader

This comment has been minimized.

Show comment
Hide comment
@dbader

dbader Jul 30, 2013

Owner

Hi @iakshay and thanks for your suggestion. I'm glad you're enjoying the workflow! The timer workflow uses the Finder's icon because of a workaround required to make notifications work at all (see issue #1). I agree that using the Finder's icon is confusing.

I looked into this again and changed the notifications to use Alfred's own icon instead of the Finder's (see 9841b57).

I don't know why I didn't do this earlier, the change was pretty simple and now it looks much nicer. So thank you for bringing this up :-) I'll package a new release soon that includes the change.

Your other suggestions look interesting as well, I'll see to them once I've got a little more time on my hands.

Owner

dbader commented Jul 30, 2013

Hi @iakshay and thanks for your suggestion. I'm glad you're enjoying the workflow! The timer workflow uses the Finder's icon because of a workaround required to make notifications work at all (see issue #1). I agree that using the Finder's icon is confusing.

I looked into this again and changed the notifications to use Alfred's own icon instead of the Finder's (see 9841b57).

I don't know why I didn't do this earlier, the change was pretty simple and now it looks much nicer. So thank you for bringing this up :-) I'll package a new release soon that includes the change.

Your other suggestions look interesting as well, I'll see to them once I've got a little more time on my hands.

@dbader

This comment has been minimized.

Show comment
Hide comment
@dbader

dbader Jul 30, 2013

Owner

I've just pushed new packages for Alfred 1 and 2 that include the icon fix.

Owner

dbader commented Jul 30, 2013

I've just pushed new packages for Alfred 1 and 2 that include the icon fix.

@iakshay

This comment has been minimized.

Show comment
Hide comment

iakshay commented Jul 30, 2013

👍

@iakshay iakshay closed this Jul 30, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment