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

Ask to receive distributed notifications immediately, and an easier way to get into the preferences #9

Closed
wants to merge 2 commits into from

Conversation

ksuther
Copy link
Contributor

@ksuther ksuther commented Dec 11, 2013

Some fixes for things that were bugging me. Thanks for making this!

…tifications sometimes getting delayed for a couple of seconds after a song change.
…access the preferences when the status item is hidden.
@citruspi
Copy link
Owner

Hey @ksuther,

Sorry about the delayed response - it was finals week and I was a tad busy.

I've merged a38281c, but I didn't merge d8cef53.

The preference window can be opened by clicking on the app icon after it's already been opened, whether the menu bar icon is hidden or not. Is this not working for you?

Have a great holiday.

@citruspi

@citruspi citruspi closed this Dec 21, 2013
@ksuther
Copy link
Contributor Author

ksuther commented Dec 21, 2013

The only way I could get the preferences window to open was through the status item. I didn't see anywhere else in the code that would open the preferences window otherwise.

@citruspi citruspi reopened this Dec 22, 2013
@citruspi
Copy link
Owner

@ksuther,

My apologies, you appear to be correct.

I'm simply surprised because I'm confident I did implement that feature some time ago in response to an email I received. I've merged d8cef53.

Either way, thank you for implementing it.

Have a great holiday.

@citruspi

@citruspi citruspi closed this Dec 22, 2013
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

Successfully merging this pull request may close these issues.

None yet

2 participants