[1/3] colorboxブランチをマージするためのマージ作業のブランチ

[omochi]

コミットを整理していく過程で、 origin/colorbox に @novi さんがコミットした、 グラフの高さ変更のコミットがコンフリクトしました。

このファイルが RenderViewControllerで、UIScrollView実装とUITableView実装の2つがありました。

コミット統合過程でのmasterブランチの編集手順は、

1. 動くScrollView実装と コメントアウトされたTableView実装
2. 動くTableView実装と コメントアウトされたScrollView実装

となっていたので、このコメントアウトされた ScrollView実装を、 RenderViewController2 として分離しました。 ( fd6d6ad )

一方、 @novi さん編集分を、 RenderViewController3として分離しました。 ( 5866e83 -> e99d4af )

その後、細かい差分が RenderViewController2と3の間にあったので、 publicを付ける方、メソッド順と空行の取り方は3の状態 に寄せるようにして、2と3を両方修正して、その2つの差分を最小化しました。 ( e553296 )

最後に、3を削除して2に上書きしました。 このコミットのdiffが、最終的に origin/colorbox からサルベージした変更という事になります。 ( ac30677 )

このPRをレビュー/マージできれば、 origin/colorbox ブランチの修正を取り込めて、 master に1本化できます。 また、ScrollView実装もコメントアウトではなく、別名クラスでの実装になるので、 コンパイラチェックがかかるようになります。

なので、 ac30677 での差分が問題ないかを、 @novi , @colorbox に見てもらいたいです。 僕が見た感じでは、 shared 周りの機能がなくなっているのと、 UIScrollView を外部から受け取るようになっていて、 動くように見えないのですが・・・。

もしかしたら、3は2の古い姿で、2で3を上書きするのが正解だったのかもしれなくて、 よくわからないです。

[omochi]

一方で、TableView実装はもう直しているので、ScrollView実装は捨ててしまっても解決します。

[omochi]

TableView実装で動いてるんですが、微妙にバグを見つけてしまったので、微妙な状況になってきました。。 2を3で上書きしましたが、そっちはデグレしてるとのことなので、本当の2を復旧しようと思っています

[novi]

さまざまな修正ありがとうございます。後ほど見てみます。

[omochi]

3から2に巻き戻した上で(d326c0a)、ScrollView実装に切り戻しました。(fd5bc05)

[omochi]

@novi さんが見てもらうのはおいておいて、お二人に確認したかったところについては @colorbox さんの回答で解決したので、僕としてはマージして良い状態になりました。

[novi]

@omochi おお、早いですねw ではマージしちゃってください。あとで気になるところがあれば、またissue上げます。

[omochi]

Its ready to merge.