shinonome-inc / qiita_client_yo

【模擬開発案件】Qiitaクライアントアプリ(PlayGroundモバイルコース最終課題)
3 stars 0 forks source link

【Flutter最終課題】ErrorView・エラー画面の実装と修正 #38

Closed KobayashiYoh closed 2 years ago

KobayashiYoh commented 2 years ago

概要

お疲れ様です。 エラー画面の実装・修正をしました。 レビューよろしくお願いします。


Client IDとClient Secret

Client IDとClient Secretが書かれているファイル(qiita_auth_key.dart)は こちらからダウンロードしてください。 ダウンロードしたファイルはmobile_qiita_app/lib/に格納してください。 よろしくお願いします。


主な変更内容

1. エラー画面をStatelessWidgetのクラスに変更

Widget型のメソッドとして作成していたエラー画面をstatelessWidgetとして作り直しました。 35ce724683af55a1bc2ab246a86adbbf02ea04ac 変更理由 : パフォーマンス向上 該当ファイル : mobile_qiita_app/lib/views/empty_serch_result_view.dart mobile_qiita_app/lib/views/network_error_view.dart mobile_qiita_app/lib/views/not_login_view.dart

2. feed_page.dartの説明変数を修正

feed_page.dart内の説明変数の定義が間違っており、検索結果が0件のときにEmptySerchResultViewが表示されないという致命的な欠陥があったため、たった1行ですが修正しました。 b65d3d023e78faae78676b3c0174694704677743 変更理由 : 説明変数を定義する際に条件式を写し間違えていたため 該当ファイル : mobile_qiita_app/lib/pages/feed_page.dart

3. アプリ内の全FlatButtonをTextButtonに変更

自分のFlutterのバージョンだとTextButtonの角を丸くすることができないと思い込んでいたのですが、自分の勘違いでした。すみません…。 FlatButtonの使用は推奨されていないため、全てTextButtonに作り変えました。その際、何度も使いまわせるようにコンポーネント化しました。 228b3ae7271f1281d5fd00c5d12ac8f369732970 d19f66d05a4ffaab7eb567be370554bfa95b39f7 変更理由 : FlatButtonの使用は推奨されていないため 該当ファイル : mobile_qiita_app/lib/components/rounded_text_button.dart

4. エラー画面に関するUIの修正

エラー画面に関するUIの修正をしました。 d19f66d05a4ffaab7eb567be370554bfa95b39f7 変更理由 : figmaとUIが少しだけ異なっていたため 該当ファイル : mobile_qiita_app/lib/views/empty_serch_result_view.dart mobile_qiita_app/lib/views/network_error_view.dart mobile_qiita_app/lib/views/not_login_view.dart

5. MyPage classの削除

MyPage classを削除してUserPage classを利用しました。 489e005bd7d33b01f8b60cd2e19ea3ccc1f2cad8 変更理由 : MyPage classが無くてもMyPageの内容を表示できるため。 該当ファイル : mobile_qiita_app/lib/pages/bottom_navigation.dart mobile_qiita_app/lib/views/not_login_view.dart


次の実装予定


実装後のUI


kota78 commented 2 years ago

丁寧に作られていて、自分が指摘できる点は無いように感じます。LGTMとさせてください。 Yoさんのプルリクの書き方、とても読みやすいです! 気になる部分があっても意図が書いてあるので分かるのがありがたいです。