search

haushinka2dx / atmosphere-web

1 stars 2 forks source link

notification permission is saved at localStorage #40

Closed kannokanno closed 10 years ago

kannokanno commented 10 years ago

Chromeで毎回確認されるのが手間だったため、 デスクトップ通知の設定をローカルストレージで持つようにしてみました。

haushinka2dx commented 10 years ago

すげぇっす。あとで見ます。

haushinka2dx commented 10 years ago

動作は :ok_hand: ソースを見る。

haushinka2dx commented 10 years ago

他はバッチリだと思います。 spec/lib/jasmine-parameterize/jasmine-parameterize.js はちゃんと見てませんが :sweat_smile:

余談ですが、通知時間とかもこれに追加していきたいですね。

kannokanno commented 10 years ago

あざます。変えました。

jasmine-parameterize.jsはサーバー側と同じもの使っているので見なくてOKです。

余談ですが、通知時間とかもこれに追加していきたいですね。

そうですねー、そんな意図も込めています。 他にもクライアント側で持てる情報(Cookie/LocalStorage/SessionStorage)はここに追加していければいいかな的な感じです。

kannokanno commented 10 years ago

例えばタイムラインの並び順とか持てるようになったらAtmosSettings.Timelineが出来てそこに持たせる的な。

haushinka2dx commented 10 years ago

対応ありがとうございます。 ソースOKです！

そうですねー、そんな意図も込めています。 他にもクライアント側で持てる情報(Cookie/LocalStorage/SessionStorage)はここに追加していければいいかな的な感じです。 例えばタイムラインの並び順とか持てるようになったら AtmosSettings.Timeline が出来てそこに持たせる的な。

良いですね。最初からサーバー側にAPI追加してTimelineの定義を保存したりするようにしようかと思ってましたが、APIのI/Fがいまいちしっくり来てなくて作りにくかったのでまずはこちらでやってサーバーAPIに反映させるのがベストですかね。

ひとまずこちらはマージしちゃいます。