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

[Notification] added OSX Notification Center #44

Merged
merged 1 commit into from
Feb 13, 2015

Conversation

evalphobia
Copy link
Contributor

This added OSX native notification feature Notification Center, other than growl.

the commit summary is below,

  • change the decision order of notifications
  • added sound parameter for OSX Notification Center

I'm not sure of my selection for the sound.
When it's noisy, I gonna find more suitable one.


MacのOSXのデフォルト通知機能である、通知センターの対応を追加しました。

  • OSXを使用していること
  • OSX Marverick以降のバージョンを使用していること
  • Growlがインストールされていないこと(whichコマンドでの判定...)

の3点を満たした場合に通知センターを使用するように、
通知機能選択の判定処理を変更しています。

失敗時の音の設定値が若干うるさく感じるかもしれませんが、その場合は他の音に修正いたします。

よろしくお願いいたします。

@iteman iteman added this to the 4.1.0 milestone Feb 13, 2015
@iteman iteman self-assigned this Feb 13, 2015
iteman added a commit that referenced this pull request Feb 13, 2015
[Notification] added OSX Notification Center
@iteman iteman merged commit ba4fca8 into piece:master Feb 13, 2015
@evalphobia
Copy link
Contributor Author

thank you for your quick action!

@evalphobia evalphobia deleted the feature/osx-notification-center branch February 13, 2015 05:38
iteman added a commit that referenced this pull request Jul 11, 2015
iteman added a commit that referenced this pull request Jul 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants