shinonome-inc / qiita_client_yukiy

2 stars 0 forks source link

[Flutter最終課題]Feature/my fix #24

Closed Yuki0316 closed 1 year ago

Yuki0316 commented 1 year ago

概要

最終テストに移る前に気になった箇所を個人的に修正しまとめています。

最終テストもまとめてこちらに記載してます。

CLIENT_IDとCLIENT_SECRET

https://drive.google.com/drive/folders/1VCIxGsXi2GFxFAxsP5GyNvWNnzs36C17 プロジェクト直下に配置してください。

スクリーンショット 2023-07-08 10 54 45

実装した内容

1. TopPageのレイアウト変更

iPhone SEでビルドした際にTopPageのレイアウトが崩れていたので、端末サイズに関わらずレイアウトが同じになるよう変更 d1c1319

2. スクロールでキーボードをClose

FeedPage内で検索をした際、スクロールと同時にキーボードが下に隠れるよう変更 b5fee46

参考記事

[Flutter]TextFormFieldのテキストをクリアするには?

3. 削除ボタンの追加

SearchBarにclearボタンを追加 ba444b0

4. 検索結果がなかった際の遷移

検索結果がなかった際に、表示させるページを追加。(ただ、最初のローディング時にもNoSearchResultが表示されてしまうので、CircularProgressIndicatorが出るように変更する必要あり) d111beb

5. 検索結果がなかった際の遷移2

検索結果がなかった際に、NoSearchResultに遷移できていなかったことが判明→修正。ローディング時にCircularProgressIndicatorを設置。(ただ位置が上になっているので中央にする必要あり) 52d8fa9

6. PullToRefresh(FeedPage)

FeedPageにrefreshIndicatorを追加し、最新記事を更新して確認できるように。検索結果がなかった際、SearchBarの中身を消去して、リフレッシュすると、記事一覧が表示されるよう変更。 3b45789

7. PullToRefresh(TagPage)

TagPageにrefreshIndicatorを追加し、最新情報を更新して確認できるように変更。 83f4f9d

8. PullToRefresh(UserPage), Dividerの設定

UserPageの投稿記事にPullToRefreshを追加、リフレッシュすると、投稿記事とユーザー情報が更新される。 FeedPageの記事とUserPageの記事のdividerの長さを条件分岐で変更。 579874a

9. ErrorPageのレイアウト修正

ErrorPageのレイアウトが少し崩れていたので修正 e6c567f 019713c

10. FetchTagの修正

ログイン時でも60回アクセスした際に403エラーが吐かれていたので、修正しました。 2b76bbe

11. Overflowエラーの修正

iPhone SE3、TopPage、認証画面、にてキーボードを表示させた後にモーダルを下げるとoverflowエラーが表示されたので修正。以下エラー時と修正後の動作画面 882a74a

before after

参考

その他

Yuki0316 commented 1 year ago

ErrorPageが表示される際に、figmaには設定されていない箇所を表示させないよう修正しました! ErrorPageをFigma通りに修正

na7ei commented 1 year ago

動作確認しました! LGTMです!

Yoshida-koshi commented 1 year ago

動作の方は問題ありません!

Yoshida-koshi commented 1 year ago

まだTextField完全に消せてないですね!

https://github.com/shinonome-inc/qiita_client_yukiy/blob/72acfdfee08503a780709c6f440a3820aeaa4838/lib/pages/user_not_login_page.dart#L18

Yuki0316 commented 1 year ago

TextFieldの修正漏れがあったので修正しました! 524fac0

Yoshida-koshi commented 1 year ago

LGTMです!マージします!