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

Notify.js によるデスクトップ通知が自動で消えない #101

Closed
akai-inu opened this issue Oct 14, 2015 · 4 comments
Closed

Comments

@akai-inu
Copy link

ソースコード上の問題を発見できず恐縮ですが、デスクトップ通知のNotify.jsが、通知後自動で消えません。

Ubuntu15.04上のOperaでの確認になりますが、 公式のDEMO 上では自動で消えることが確認出来たため、実装の問題なのではないかと思います。

毎回クリックするのが面倒なので修正して頂ければ幸いです。

(通知をクリックしたら該当のKnowledgeページに飛んでくれるとさらに嬉しいです 🐈)

@akai-inu akai-inu changed the title Notify.js の通知が自動で消えない Notify.js によるデスクトップ通知が自動で消えない Oct 14, 2015
@koda-masaru
Copy link
Contributor

ご連絡ありがとうございます。

Notify.js の オプションで timeout が指定できるのですが、今のところ設定していないので、
通知が消えないようになっています。

たしかに毎回クリックするのは面倒ですので、3秒程度で消すように設定しようかと思います。

@koda-masaru
Copy link
Contributor

後で対応するため、調べた事を記載します。

/knowledge/src/main/webapp/WEB-INF/views/commons/layout/commonScripts.jsp

に、Notify.js を使った部分が書かれているので、以下の部分を修正する。

  • 修正前
                var myNotification = new Notify('Notify from Knowledge', {
                    body: msg,
                    notifyShow: onNotifyShow
                });
  • 修正後
                var myNotification = new Notify('Notify from Knowledge', {
                    body: msg,
                    notifyShow: onNotifyShow,
                    timeout: 3,
                    notifyClick: onNotifyClick // クリックしたら Knowledgeのページへ遷移する
                });

@koda-masaru koda-masaru self-assigned this Oct 19, 2015
@koda-masaru koda-masaru added this to the v0.6.0 milestone Oct 19, 2015
koda-masaru added a commit that referenced this issue Oct 20, 2015
…desktop

#101 デスクトップ通知は3秒で消えるようにして、通知をクリックするとそのナレッジを表示するように変更
@koda-masaru
Copy link
Contributor

通知は3秒で消えるようにしました。
また、通知をクリックすると、通知されたナレッジを表示するようにしました。
次回のリリースをお待ち下さい。

@yamayuski
Copy link

ご対応ありがとうございます!

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

No branches or pull requests

3 participants