sfc-rg / rg-portal

Internal Website for Keio Univ. Technology joint research groups
https://portal.sfc.wide.ad.jp/
12 stars 5 forks source link

Refactor index page #72

Closed miyukki closed 9 years ago

miyukki commented 9 years ago

pre_built_pages#topをなくしました。目的は将来的にこのコントローラーを無くしたいです。

miyukki commented 9 years ago

@sfc-rg/reviewer レビューお願いします

miyukki commented 9 years ago

@demmys

demmys commented 9 years ago

@miyukki PreBuiltPagesControllerを無くしたいというのは理解できなくもないけど、どこに吸収していく?

pre_built_pages#topを作ってたのは、pages#indexはその意味合い的にページの一覧(サイトマップ的なもの)を表示するじゃないかなと思ってたからなんだけど...

miyukki commented 9 years ago

@demmys 返信がおそくなった。 pages#indexがページ一覧的な役割を持つかは、いまのところまだ未定なのでおいておく。 PreBuiltPagesControllerという感じの特殊な用途を出来るだけ実装に落としたくない(WipTermとかThesisとか限定されるし、また同じようなものを作るんヒア新たに作らなくてはいけない) (最初にRG PortalはRG内のロジックのコードが入っていても良いといったけど

なので、そのうち

{include_content path=/thesis}

みたいなインクルードをするようなスニペットタグを作ったり、するのがベストかなぁと思っている。 けど今のとこぱっとした答えは出てない。

demmys commented 9 years ago

@miyukki あー、それいいね。そういう独自のマクロ的な形で組み込むほうが確かにきれいだ。