search

ViBii / mitemiru

2 stars 0 forks source link

[IR] グラフレイアウトの調整 #327

Closed okjsc1118 closed 8 years ago

okjsc1118 commented 8 years ago

Overview

326 の実装

Task

グラフの描画領域の縦幅はデータ数に合わせて可変します。 横幅は1080pxに固定します。

Review

グラフの位置を中心に寄せました。 実際に動かして確認してください。

違和感等あればコメントください。

okjsc1118 commented 8 years ago

レビューお願いします。

sjc1209 commented 8 years ago

image 処理する時、グラフ間の空白がありますが、これで大丈夫？

okjsc1118 commented 8 years ago

@sjc1209 それは良くないやつだな。 どこのグラフの間で起きてるの？

sjc1209 commented 8 years ago

生産性グラフとコメント数グラフの間です。

okjsc1118 commented 8 years ago

@sjc1209 状況も詳しく教えて 読み込み終わっても空いたまま？

sjc1209 commented 8 years ago

私たちのRedmineの認証情報で登録したプロジェクトを選択する時、待っている間、生産性グラフとコメント数グラフの間こんな感じが出た

sjc1209 commented 8 years ago

image この図はちょっと真ん中じゃないような気がするけと

okjsc1118 commented 8 years ago

@sjc1209 Chrome使って！

sjc1209 commented 8 years ago

なるほど、

okjsc1118 commented 8 years ago

@sjc1209 あとコミット数のグラフはソートボタンがあるから若干左よりです

sjc1209 commented 8 years ago

image 生産性グラフ棒グラフになった時、sortの文字とSUPPORTの文字の距離がちっと近いですね。

sjc1209 commented 8 years ago

image ここも同じような感じ。。

okjsc1118 commented 8 years ago

@sjc1209 全体の画像見せてもらえる？

sjc1209 commented 8 years ago

image

sjc1209 commented 8 years ago

後は今のコメント数のグラフは他のグラフより、小さいですが、ヒートマップの大きさは変えますか。

okjsc1118 commented 8 years ago

@sjc1209 生産性の棒グラフ修正したので確認よろしく。 ヒートマップは人数すくなければ小さいのはしょうがないかな。 大きくもできるけど、あんまり意味はないと思う。

sjc1209 commented 8 years ago

なるほど、確認しました。OKです。

okjsc1118 commented 8 years ago

マージしちゃいますー