画面への遷移追加

tourist-project / yuwaku_proto_flutter

湯涌用アプリ

1 stars 0 forks source link

Closed keigo194547 closed 2 years ago

keigo194547 commented 2 years ago

以前までついていなかったtopViewから遊び方画面への遷移機能の追加をしました。問題なかったらマージしてください。

yamanetaisei commented 2 years ago

@keigo194547

動作確認しました！！いい感じ:+1:

ひとつ気になったんだけど、画面のしたのbottomTabがあるなら、わざわざtopViewからボタンで画面遷移できるようにする必要はないのかなぁと思ってるけど、そこら辺はどう考えてる？？

僕個人的にはどちらか一方でいいと思ってる。

bottomTabだけでは情報が少なくてユーザーに優しくないって思うなら、bottomTabは無しでボタンだけの遷移にしちゃえばいい気がする。

yamanetaisei commented 2 years ago

ここで、全体の使用に関する議論を2人でするのも野暮なので、このPRは取り込んじゃいますか

LGTMです

LGTM

keigo194547 commented 2 years ago

@yamanetaisei BottonTabBarとtopViewの2つを採用している理由としては

との意図があります。もし、どちらか一方を採用するのであればBottonTabBarの下にページの文言などを加えるとかがいいですかね？

yamanetaisei commented 2 years ago

@keigo194547

ありがとう！！

何度か画面遷移した人に対して、いちいちtopViewに戻らなくてもいいように

確かにそれはその通りだなぁと思った。

それよりもissue #62 の下のタブバーがボタン遷移だと表示されないことの方が違和感があるから、そっちを修正した方がいいなって思った。

ひとまずこのままで、#62 に着手する感じで進めていきますか！！

keigo194547 commented 2 years ago

@yamanetaisei DUBnkWuV4AApLFp