-
Notifications
You must be signed in to change notification settings - Fork 0
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
設定画面モーダル作成 #5
設定画面モーダル作成 #5
Conversation
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.
コメントしました。
SettingsPageの変更が差分に含まれていないので修正お願いします🙏
static const TextStyle h3BasicBlack = TextStyle( | ||
fontSize: 12, | ||
color: AppColors.black, | ||
fontWeight: FontWeight.w500, | ||
); | ||
static const TextStyle h1ThinBlack = TextStyle( | ||
fontSize: 17, | ||
color: AppColors.black, | ||
fontWeight: FontWeight.w100, | ||
); |
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.
スタイルの定義がデザイナーさんっぽくていいですね👍
class AppBottomModalSheet extends StatelessWidget { | ||
final String title; | ||
final Widget content; | ||
final bool showBottomDivider; |
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.
FeedPageでの使い回しを意識できていていいですね👍
child: Container( | ||
padding: | ||
const EdgeInsets.symmetric(horizontal: 16, vertical: 18), | ||
child: SingleChildScrollView( | ||
child: content, | ||
), | ||
), |
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.
プルリクに添付された画像だけだと気づけなかったのですが、画面上部の余白も切れているので今後修正お願いします。
これ以上長引いてもアレなので、今回はマージします。
class Texts { | ||
static const String privacyPolicyText = ''' |
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.
コンポーネント化もしっかりできていて綺麗なコードだと思います!!LGTMです🙆
LGTMです |
概要:設定画面モーダルを作成しました
CLIENT_IDとCLIENT_SECRET
実装した内容:設定画面モーダルを作成しました
UI
動作確認
参考
その他