Side drawer

[keigo194547]

サイドバーの実装

[keigo194547]

@KitagawaFuuga 時間があったら手元で確認してみてください．特に問題なければマージお願いします．

[yamanetaisei]

@keigo194547 さらっと確認しました！ 必要なさそうな改行がいくらかありそうなので :eyes: のコメントしときました（何個か見落としてるかも）

UIの差分がある時はスクショとかもらえるとうれしい！！

差分を見ると、今回サイドメニューの他にColorもいじってるぽいので、そのこともPRの概要とかに追記しておくべきかなと思います！（本来その差分を混ぜてもいいかは別として。。。）

[keigo194547]

ひとまずWebサイト遷移するメソッドの処理を分けた(ファイル分けはしていない) あとAndroidStudioがiOSの実機を認識してくれない...

[yamanetaisei]

@keigo194547 修正助かります 🙇

ひとまずWebサイト遷移するメソッドの処理を分けた(ファイル分けはしていない)

ファイル分けはした方がいいかなと思う。もともとこのファイルはsome_top_page.dartっていう名前からも何となく伝わる通り、topページに関することが書いてあって、それとWebサイトの遷移処理は全く別ドメインの話なので分けるべきかなと。

あと、Webサイトの遷移が他の画面でやることになった場合、その画面が定義されてるファイルにimport 'package:yuwaku_proto/homepage_component/some_top_page.dart';っていうふうにしないといけなくて、本来必要のないトップページの情報まで参照できるようになってしまうからそこは関心を分離しておきたい感じはする

[yamanetaisei]

あとAndroidStudioがiOSの実機を認識してくれない...

修正が済んだら自分の方でしておきます

[keigo194547]

• drawerレイアウトコード(Widget)の分離
• 各ファイル作成

[yamanetaisei]

@keigo194547 1点だけ気になったところをレビューしました！

あとはFigmaのデザインに寄せてもらえると助かります（画像の差し替えとか）

変更してくれたヘッダーの色は今回はこのPRに含めて大丈夫です！！

[keigo194547]

ロゴ変更と命名修正しました．

[keigo194547]

@yamanetaisei サイドバーの背景色変更もこのブランチでやってもいいですか？

[yamanetaisei]

@keigo194547 サイドバーの背景だけ変えるってことだよね？ 今回のタスク以外に影響が及ばないなら大丈夫だと思われ

[keigo194547]

Sidebarの色変えました

[keigo194547]

@yamanetaisei @KitagawaFuuga 確認よろしくです．

[yamanetaisei]

@keigo194547 コードはOKだと思います！！お疲れ様でした！！！ iOSの動作確認もできました！！

利用規約がまだないのはわかりますが、他の埋め込まれてるURLが正しいかは自分の方では判断できないので関谷くんにみてもらった方がいいかも。（ひょっとしてどこかにまとめてある？？）

動作の画面録画したのでSlackで送ります使いたければ使ってください.

それの確認が取れたらマージしようと思います。

[yamanetaisei]

@keigo194547

Slackでメンション飛んでると思うけど、↓こちら対応お願いします。それができたらマージします

https://tourism-63q6320.slack.com/archives/C034P5EPW73/p1661839759112969?thread_ts=1661783675.534319&cid=C034P5EPW73

[keigo194547]

URLの修正したしました〜

[yamanetaisei]

正しいURLが差し込まれてることを確認しました。 LGTMです 👍