kazukitash/demo_textbook — pull request #1 by kenohata (2013-12-23 09:03:04+00:00)
Title: コード読んだ

コード読んで気になった所を勝手に修正しました。

* FormViewをNewViewにリネームした。EditとNewを共通化してFormと呼ぶのであればこの命名に意義は無いが、そういう感じでも無かったから。
* Collectionが重複していたので片方削除した。
* AddFormViewは、Viewを作るほどの仕事をしているとは思えなかったので、IndexViewにその機能を持たせることにした。Viewの存在価値は、eventsハッシュまたはModelのイベント監視のある程度の粒度ごとに切り出す点にあり、Templateをrenderする物として用いるにはコストが高くて無駄が多いと思う。
* NewView(旧名FormView)の送信ボタンがaタグで実装されていた点が気になったので、buttonタグで実装しなおした。こうすることで、Tabキーでカーソルが勝手にbuttonに移動するようになった。この挙動はブラウザの性質。

いちおう俺のこのプルリクのブランチをcheckoutしてから動作確認とコード変更点を確認してからマージ、という流れでよろしく。