[Session/1] 天気予報画面の UI 作成 - Githubissues

daichikuwa0618 / flutter-weather-app

Dummy Weather App Built with Flutter

1 stars 0 forks source link

[Session/1] 天気予報画面の UI 作成 #14

Closed daichikuwa0618 closed 6 months ago

daichikuwa0618 commented 6 months ago

課題

close #2

対応箇所

[x] Placeholder の幅は画面の幅の半分
[x] 青字と赤字の Text の幅は Placeholder の幅の半分
[x] Placeholder の高さと幅は同じ
[x] Text の上下に 16 logical pixel のパディング
[x] Text の文字は ** ℃
[x] Text の水平位置は中央
[x] Text のスタイルは Theme.of(context).textTheme で取得してきた TextTheme の labelLarge
[x] Text の色は左が Colors.blue、右が Colors.red
[x] Placeholder の水平中央は画面の中央と同じ
[x] Placeholder と Text を合わせた矩形の垂直中央は画面の中央と同じ
[x] Text と TextButton の隙間は 80 logical pixel
[x] Text と TextButton の水平中央は同じPlaceholder の幅は画面の幅の半分
[x] 青字と赤字の Text の幅は Placeholder の幅の半分

動作

expected	actual (iOS)	actual (Android)
#2 参照

github-actions[bot] commented 6 months ago

Ready for review :rocket:

daichikuwa0618 commented 6 months ago

~~check-pr は通るけど flutter test の report が来てない、、、？~~

来た

daichikuwa0618 commented 6 months ago

Approve されていて、追加の修正も明瞭に指示いただけたおかげで追加のレビューは不要と思っています。 conversations は自分の方で resolve させてもらって、マージしたいと思います。

マージ後でもなにかお気づきの点があればご指摘ください。