search

haushinka2dx / atmosphere-web

1 stars 2 forks source link

Timeline is displayed in order of the past from the newest. #2

Closed kannokanno closed 11 years ago

kannokanno commented 11 years ago

magic swashIn を最新から適用するようにしました。 修正自体は2行ですけど、プルリク-マージの流れの確認も兼ねてプルリクにします。 (multicolumnにプルリクでいいんですよね？)

kannokanno commented 11 years ago
haushinka2dx commented 11 years ago

multicolumn にプルリクで問題ないかと思います。 （そのうちmasterに反映します）

本題ですが、現在の実装にしている理由としては、 swashIn を適用というよりはタイムラインのアイテム自体を過去から最新に表示（170行目の $(item).show(); ）するようにしていて、 それは時系列順に段々と表示された方が思考的に自然かなと思ってそのようにしています。

現在の逆順にすることが間違いと思っていませんが、 その意図は何でしょうか。 （初期表示で全部表示されるまでが長いせいかなと思っていますが）

なお、コード的には問題ないかと思います。

haushinka2dx commented 11 years ago

ローカルで反映して確認してみましたがそんなに違和感ないですね。 それよりは「NewItem」をしばらく強調表示するほうが重要のように思えてきました。

意図だけ確認して問題なければ取り込みたいと思います。

本来は意図とかはIssueでやるべきですかね？

kannokanno commented 11 years ago

その意図は何でしょうか。 （初期表示で全部表示されるまでが長いせいかなと思っていますが）

お察しの通り表示までの時間が長いためです。 現状 everyone's message の最新が見えるまで7秒近くかかってしまいます。 最新を確認したい場合に、これは不便でした。 数が少ない messages to you は1-2秒で、こちらは特に違和感ありません。

今はページング(ajax loading)がないから200件までがっつり取ってくるのかなと思っています。 ページング対応したら一度の取得件数が減らせられると思いますので、過去から適用に戻しても問題ない気がします。

本来は意図とかはIssueでやるべきですかね？

今回はここで問題ないと思います。

などはissueにした方があとあと見やすいかもしれません。 厳密に決めず、ゆるふわに適当な判断でよいでしょうけども。 Githubの検索使えば探すのも簡単でしょうし。

haushinka2dx commented 11 years ago

意図了解しました。 この後マージします。

あとゆるふわも了解です。