Mayumi-Nakabayashi / withTone

2 stars 0 forks source link

Feature/setting profile #16

Closed sodateya closed 1 year ago

sodateya commented 1 year ago

やったこと

設定とプライバシー画面作成 ・SingleChildScrollViewに各Widgetを設置

プロフィールを編集作成 ・SingleChildScrollViewに各Widgetを設置 ・写真を変更、動画を変更のUIはGestureDetectorをStackにて背景に画面、表面にアイコンを設置

やってないこと

・各Widgetに入る要素を変数で管理していない

相談

1.各ページのListTile、を今後増やすことを見越してコンポーネント化すべきか 2.各ページの線で区切られた部分をそれぞれListViewにするべきか 3.プルリクエストってこんな感じで良いですか,,,?

UI

Figma(setting) Flutter(setting) Fingma(edit_profile) Flutter(edit_profile)
setting Simulator Screenshot - iPhone 14 - 2023-08-21 at 09 58 05 edit_profile Simulator Screenshot - iPhone 14 - 2023-08-21 at 09 58 08
ken-ty commented 1 year ago

ご対応ありがとうございますー! PR こんな感じで良いと思います、見やすいです 👍

yuuya-1205 commented 1 year ago

相談、ありがとうございます!

1.各ページのListTile、を今後増やすことを見越してコンポーネント化すべきか

→コンポーネントして良いと思います! →他のページでもprofileページ以外でも使われているので! @ken-ty @y129may9th 意見いただければ嬉しいです!

3.プルリクエストってこんな感じで良いですか,,,?

→すごくわかりやすくていいと思います!こんな感じでやっていきましょう!

ken-ty commented 1 year ago

@sodateya cc: @yuuya-1205 @y129may9th 1.各ページのListTile、を今後増やすことを見越してコンポーネント化すべきか → はい, お手隙の際にやっていただけると! 時間都合つかなかったら, 後回しでも, 別 PR 作って対応でも大丈夫ですよ! 3.プルリクエストってこんな感じで良いですか,,,? → いい感じです! 次の MTG で commmit message の書き方共有しますね!

ken-ty commented 1 year ago

LGTM! こちらの差分 #22 で使いたいので, 一旦マージして 別 PR でコンポーネント化やっていただけると嬉しいです...!!