Skip to content

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

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

Merged
merged 50 commits into from
Sep 29, 2019
Merged

Conversation

yinm
Copy link
Contributor

@yinm yinm commented Sep 22, 2019

resolve #134

レビューポイント

  • グローバルメニューを作成する #134 (comment) にあることが実現できているか
  • 他にスタイルで気になるところがないか (レスポンシブにするためのサイズ指定が怪しそうな感じがしてます...)
  • JSの実装 (スクロール検知・メニューを開いている時はスクロールさせない)できになるところがないか

@yinm yinm self-assigned this Sep 22, 2019
@yinm yinm added WIP and removed PR: unreviewed labels Sep 22, 2019
合わせて、メニューを開いた状態でロゴを押した時に、メニューを閉じれるようにする
@umiremix
Copy link

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

@yinm
Copy link
Contributor Author

yinm commented Sep 29, 2019

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

Copy link
Contributor

@inouetakuya inouetakuya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

上下中央揃えについて LGTM です!他の方が OK だったらマージしちゃってください〜

image

Copy link
Contributor

@448jp 448jp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
ミサワ

@umiremix
Copy link

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

@yinm
Copy link
Contributor Author

yinm commented Sep 29, 2019

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

Copy link
Collaborator

@ryamakuchi ryamakuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM です!シュッと対応ありがとうございますー!! 🙇‍♀
lgtm

@yinm
Copy link
Contributor Author

yinm commented Sep 29, 2019

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

@yinm yinm merged commit 4b40e03 into master Sep 29, 2019
@yinm yinm deleted the add-global-navigation branch September 29, 2019 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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