グローバルメニューを作成する

yinm commented 5 years ago

レビューポイント

https://github.com/kazupon/vuefes-2019/issues/134#issuecomment-529292406 にあることが実現できているか
他にスタイルで気になるところがないか (レスポンシブにするためのサイズ指定が怪しそうな感じがしてます...)
JSの実装 (スクロール検知・メニューを開いている時はスクロールさせない)できになるところがないか

yinm commented 5 years ago

@umiremix @inouetakuya @ryamakuchi @aytdm @448jp お待たせしました！レビューをお願いします 🙏

inouetakuya commented 5 years ago

@yinm ひと通りコメントしましたので確認お願いします〜

感想）沖さんのレビュー、めっちゃ勉強になる 📝

448jp commented 5 years ago

私も井上さんのレビュー、いつも大変勉強になってますYo！（異文化交流）

umiremix commented 5 years ago

@yinm 実装ありがとうございますー！今日中に確認致します！:pray:

umiremix commented 5 years ago

@448jp 事前に相談せずにすみません…。クリック後にスムーススクロールを入れたほうがアンカーリンクだとわかりやすいかと思ったのですが、いかがでしょうか？

448jp commented 5 years ago

@umiremix はい、私もそう思って本プルリクにコメントしてます！👌 https://github.com/kazupon/vuefes-2019/pull/181#discussion_r327086415

umiremix commented 5 years ago

@448jp あああああすみません、見落としてました…！レビューありがとうございます！ほぼ気になる点は沖さんの方でレビューいただいておりました！:pray::sparkles:

@yinm 遅くなりすみません、一通り確認致しました！ご対応よろしくお願いいたします。

yinm commented 5 years ago

@inouetakuya @448jp @umiremix レビュー反映がまだ完了していないのですが、明日中には対応しますので、しばしお待ちを... 🙏

yinm commented 5 years ago

@inouetakuya @448jp @umiremix レビュー反映したので、再度レビューをお願いします！ :octocat:

448jp commented 5 years ago

@yinm @inouetakuya

上下中央に来るように、再度微調整をお願いします〜 🙏 （あともうひと息だ！

.togglable-menu に height を与えた上（逆に .menu-button に height は要らないかも？）で以下にすれば上下中央に来るかと思います！

top: 0;
bottom: 0;
margin: auto;

umiremix commented 5 years ago

すみません、前回レビューしたところsubmitできてませんでした…！:sob: 可能な範囲でご対応いただけますでしょうか…

yinm commented 5 years ago

@inouetakuya @448jp @umiremix レビューありがとうございます〜！レビュー反映したので、もう一度お願いします！

umiremix commented 5 years ago

LGTMです！！ナイスな実装ありがとうございました:pray: 勉強になりました〜！ 30381

yinm commented 5 years ago

@inouetakuya @ryamakuchi 以下の対応をしたので、レビューをお願いします 🙏

feat: スポンサー一覧ページを作成する by ryamakuchi · Pull Request #170 · kazupon/vuefes-2019 と nuxt.config.tsがコンフリクトしたので解消した
smooth scrollを実現するpluginを汎用的なもののみにした (sponsors以外のページでも使えていい処理だと思うので、ページを限定する処理を外しました)

yinm commented 5 years ago

みなさんありがとうございます〜！マージします！