search

pj-picbook / picbook

9 stars 0 forks source link

Feature/#50 fab navigation #96

Closed Ta23ka98 closed 2 years ago

Ta23ka98 commented 2 years ago

関連のタスクissue*

PR作成時のチェックリスト*

※対象外の場合は実施していなくてもチェックを入れる

対応したこと*

・FABに検索画面への画面遷移を実装した ・ボトムナビゲーションに検索画面への遷移機能があったが、不要となったため削除した。

スクリーンショット 2022-08-15 12 25 11
Ta23ka98 commented 2 years ago

@karokita あまり関係ないのですが、本棚画面の上にスマホの情報(バッテリーやネットの接続状況)が出なくて大丈夫でしょうか？ (たぶんMaterialAppでWrapしてない等...？🤔)

karokita commented 2 years ago

@Ta23ka98 たしかにですね！ 他画面も表示されていなかったので、全体的に確認するタスクとしてこちらにつみました　https://github.com/pj-picbook/picbook/issues/69

tokku5552 commented 2 years ago

こちらって関連タスクは #50 ですかね？

Ta23ka98 commented 2 years ago

@tokku5552

こちらって関連タスクは https://github.com/pj-picbook/picbook/issues/50 ですかね？

すみません、誤解していました🙇 てっきり要件や、親ストーリーで参照されているissueを関連タスクだと思っていました。

これからは対応したissue自体を書くことにします🙋

tokku5552 commented 2 years ago

特に問題なくマージできてそうなので大丈夫だと思います！

Ta23ka98 commented 2 years ago

@tokku5552 確認ありがとうございました！🙌