プロジェクト

全般

プロフィール

Defect #156

完了

ファイルの行数が多い場合に、ファイル内容の表示に時間がかかる

Tatsuya Saito さんが14年以上前に追加. 14年以上前に更新.

ステータス:
終了(Closed)
優先度:
通常(Normal)
担当者:
対象バージョン:
開始日:
2009/08/16
期日:
進捗率:

100%

予定工数:

説明

リポジトリ上のファイル内容を、[見る]や[注釈]などで確認する際、ファイルの行数が多い(1000行など)場合に非常に時間がかかってしまいます。
また、ある程度時間が経過すると、ブラウザからJavascriptを実行を停止するか確認するダイアログが表示されます。

再現手順は以下のとおりです。
  1. 1000行以上のファイルをリポジトリにコミットする。
  2. [リポジトリ]タブをクリックする。
  3. [最新のリビジョン]からコミットしたリビジョンを選択する。
  4. コミットしたファイルを選択する。
確認した環境は以下のとおりです。
  • CodeReview Plugin r300
  • Firefox 3.5.2(Win) と IE8

上記の手順にあるとおり、ファイルを選択した時点でだいぶ待たされた後、ブラウザから警告されるような状態です。
[注釈]の場合のみレビュー可能にするなど、ユーザーがワンクッションおけるようにしたほうがよいのではないかと思います。

Haru Iida さんが14年以上前に更新

  • ステータス新規(New) から 解決(Resolved) に変更
  • 進捗率0 から 100 に変更

チェンジセット r302 で適用されました。

Haru Iida さんが14年以上前に更新

報告ありがとうございます。
Javascriptを見直して処理時間を短縮してみました。
こちらでは2000行程度は大丈夫でした。
これで試してみてもらえますか?

Haru Iida さんが14年以上前に更新

  • 担当者Haru Iida にセット
  • 対象バージョン0.2.3 にセット

Tatsuya Saito さんが14年以上前に更新

私の環境でも2000行のデータではFirefox、IE8ともに警告はされなくなりました。
ありがとうございました。

以下に関しては、行数がもっと多い場合も考慮してあったほうがよいかと思いますがどうでしょうか?ユーザーの環境にも寄りますし。
bq. [注釈]の場合のみレビュー可能にするなど、ユーザーがワンクッションおけるようにしたほうがよいのではないかと思います。

Haru Iida さんが14年以上前に更新

確認ありがとうございます。

Tatsuya Saito wrote:

以下に関しては、行数がもっと多い場合も考慮してあったほうがよいかと思いますがどうでしょうか?ユーザーの環境にも寄りますし。

確かにそうですね。人によってはチープなマシンで1万行のソースを見なきゃならない場合もあるかもしれませんね。まあ、必要にせまられたら実装することにします。YAGNIの原則。

Javascriptはもうちょっとチューニングする余地がありそうなので5000行ぐらいは表示できるようにはなるかなぁ。それ以上は、1ファイルにそんなにコードを書いちゃいけませんということで(^^

Haru Iida さんが14年以上前に更新

  • ステータス解決(Resolved) から 終了(Closed) に変更

他の形式にエクスポート: Atom PDF