copetit
commented
2 years ago やったこと
- ボタン同士に同じclassNameを使ってるのが多かったのでなるべく統一させました。
- ロゴにメイン画面へのリンクを入れました。(今後posting画面にもヘッダーを入れるからです。)
- 位置情報を提供しない場合、デフォルトで東京駅が表示されるようにしました。メイン、posting画面両方です。 (位置情報の許可を聞く窓が表示されるときに、loadingを表示させたい気持ちですね。それは一旦TODOに残しました。)
確認方法
ボタン
ソースコードの修正が多いので、全てのボタンが正常に表示されて正常に動作するかだけ見ればいいかなと思います。
ロゴ
メイン画面のロゴをクリックするとメインに戻ること
位置情報
プライベートブラウザを開いて以下を確認。
- 「遮断」を押すとマップが東京駅になること
- 「許可」を押すと現在の位置が表示されること
こちら両方確認しましょう。
shhwan
やること
ボタンのクラスがかなり共通なのでパーツ化します。
atomic designを採用するのも考えてみる
こちらのissueが買ったんだったのでこのissueに載せて修正します。 https://github.com/copetit/whereareyou/issues/132
参考URL