Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Menu footer #55

Closed
wants to merge 11 commits into from
Closed

Menu footer #55

wants to merge 11 commits into from

Conversation

kamataryo
Copy link
Contributor

menu-header

@kamataryo
Copy link
Contributor Author

kamataryo commented Mar 6, 2017

#23 に対して、main-contentsを流用して作った人を表示するメニューを追加しました。
opts.titleに作った人の名前を、 opts.contentsに自己紹介を流し込んで表示しています。

@kamataryo
Copy link
Contributor Author

main-contentsにitemsが表示されているのか、ホームコンテンツが表示されているのか、などの状態管理の仕組みを作った方がいいでしょうか?

@kamataryo kamataryo requested a review from miya0001 March 6, 2017 13:00
@miya0001
Copy link
Contributor

miya0001 commented Mar 6, 2017

いいですね。
他の項目と区別するために下の方に固定するといいかも。(ホームに戻るためのボタンがじゃまになりますが、それを別のとこに持ってくといいかも。)

ただ、入れるべきコンテンツは、いろいろ好みがありそうなので、以下のような感じでシンプルにタグを一つ置いとくだけでいいかもです。
https://github.com/wakayama-hacker/wnew/blob/master/tags/menu-header.tag
このファイルに好きなようにHTMLで書いてって感じで。

@kamataryo
Copy link
Contributor Author

kamataryo commented Mar 7, 2017

TODOs

  • menu-footerはメニューエリアの下部に固定(ただし、カスタマイズ要素のため、デフォルトは空)
  • config.jsonで求めるauthorsの情報を精査して整理する
    • 名前
    • GitHubアカウント名
    • URL
    • 自己紹介的なもの
  • ホームコンテンツ直下に置いたauthorsセクションのhtmlを整理してスタイルを当てる

@kamataryo
Copy link
Contributor Author

現在はこんな感じです。
wnew

@miya0001
Copy link
Contributor

miya0001 commented Mar 7, 2017

あっ、サイドメニューのフッターにaboutみたいな項目を表示して、それをクリックしたらREADMEが表示されるようにするといいかも。
トップページはhome.mdっていうファイルから読み込むようにすればデモ用のアプリの見た目をもうちょいよくできるかもですね。

どう思います?

@miya0001
Copy link
Contributor

miya0001 commented Mar 7, 2017

#40
先にこれやります。
最終的に #/about みたいなURLでコンテンツが表示されるといいかも

@kamataryo
Copy link
Contributor Author

デモ用のhome.mdを作っておきます。

@kamataryo
Copy link
Contributor Author

kamataryo commented Mar 7, 2017

固定ページ生成のGulpタスクを作ることにして、home.mdabout.mdなどのファイルから対応するタグファイルを自動生成するようにしたいと思いますが、いかがでしょうか。

@miya0001
Copy link
Contributor

miya0001 commented Mar 7, 2017

ちょっと考えます。ドキュメントの記述を簡単にできる方法ないかなと。

@miya0001
Copy link
Contributor

miya0001 commented Mar 7, 2017

これを実装することは、すくなくともこれをフォークするプロジェクトではこの部分をカスタマイズすることが必須になるんですよね。
フォークする時にカスタマイズが必要な範囲が少ないに越したことはなくて、ここはカスタマイズできるけどめんどくさければしなくてもいいよという感じにしたいのです。

@miya0001
Copy link
Contributor

miya0001 commented Mar 7, 2017

「やりたければやれば?でもめんどくさいならやらなくてもいいよ」という選択肢を残したい感じっす。
他のアプリをいろいろ見たんですが、実際 About みたいなものがないアプリが多いんですよね。
なのでそれを不要と思う人は、無視しても問題ないようにしてあげたいなと。

@kamataryo
Copy link
Contributor Author

今イメージしているのは、例えば./dataフォルダに about.mdが入っている時だけ abount-contents.tagが生成されて、アプリ本体ではこのような動的生成されたタグは動的にrequire・mountされる感じです。

@miya0001
Copy link
Contributor

miya0001 commented Mar 8, 2017

ルーター機能をつけました。

https://github.com/wakayama-hacker/wnew/blob/master/src/app.js#L110-L133

router( 'about', function( div ) {
  riot.mount( div, 'about' )
} )

みたいな感じでできるので、とりあえずこの機能はなしにしたいと思います。
すいません。

@miya0001 miya0001 closed this Mar 8, 2017
@kamataryo kamataryo deleted the menu-footer branch March 9, 2017 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants