-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#71 仕事詳細ページの実装 #88
#71 仕事詳細ページの実装 #88
Conversation
・JobDetailPage._buildSection => SimpleSection ・JobDetailPage._buildHostTypeChips, JobDetailPage._buildAccessTypeChips => RowChips
final Iterable<T> allData; | ||
|
||
/// 選択肢をキーにしたデータのラベル | ||
final Map<T, String> allLable; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[タイポ] もしエディタにタイポチェックの機能が入っていなければぜひ入れることをおすすめします!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あと、変数名は単に labels
でもいいかも?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ご確認ありがとうございます!
修正しました。
タイポも入っていませんでしたので追加しました!
return Scaffold( | ||
appBar: AppBar( | ||
title: const Text('お手伝い募集'), | ||
centerTitle: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
中央に配置するなら一括で MaterialApp の appBarTheme でやりたいので、ここは一旦消しましょうか!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
一旦削除しました!
とりあえずテーマは変更していませんが、今回のPRでテーマも変更しますか?
もしくは幾つか画面完成後にデザインをみて決定という形でしょうか?
Widget _buildErrorPage(String message) { | ||
return Center( | ||
child: Text(message), | ||
); | ||
} | ||
|
||
Widget _buildLoadingPage() { | ||
return const Center(child: CircularProgressIndicator()); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この 2 つはいったん必要な箇所にべたっと書いてもらっていいです!(こういうエラーが起きた場合系の UI はあとで一括でうまくやる方法考えたいです!!うっすら考えておいてもらえたりするとめちゃめちゃ嬉しいです!!)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
修正しました!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ちょっとリファクタ追加して、Approve しました!!88eba3e
今回もとてもすばらしい実装・PR ありがとうございました!!
Issue
#71
close #71
説明
仕事詳細ページを実装しました。
UI
その他
なし
チェックリスト