UIラベルアプリ提出

shinonome-inc / mobile_sakai

0 stars 1 forks source link

Closed yasyun closed 3 years ago

mcz9mm commented 3 years ago

土台の部分がうまくできていないので修正お願いします。参考リポジトリを見ればわかるので自分の実装と差分を確認してください。

主に、・ラベルの横幅は数値固定を用いない(width=370とか)→AutoLayoutで左右に制約から16pt等つける・AlignCenterX, Yで数値をずらすのではなく、一つのラベルを基準に制約をつける

ShuheiYoshidaJP commented 3 years ago

まず参考リポジトリのUIをコピペしてその上で大きさを整えたり、色をつけたり、フォントをいじったりしてもらうのがいいかと思われます。

ShuheiYoshidaJP commented 3 years ago

・ラベルの横幅は数値固定を用いない(width=370とか)→AutoLayoutで左右に制約から16pt等つける・AlignCenterX, Yで数値をずらすのではなく、一つのラベルを基準に制約をつけるこの2点は反映されていることが確認できました。

mcz9mm commented 3 years ago

制約のつけ方がちょっと気になりました。左右は16ptなのに上下のラベル同士の制約が20.67ptとか46.667とかになっているのかな。整数値で統一して実装しましょう。

ShuheiYoshidaJP commented 3 years ago

すみません💦 RUNさせずにmain.storyboardだけを確認していました。コードで設定していたんですね。せっかくなので良い勉強にもなると思うのでstoryboard側で色、角丸、大きさなどを設定してみましょう。

yasyun commented 3 years ago

制約のつけ方がちょっと気になりました。左右は16ptなのに上下のラベル同士の制約が20.67ptとか46.667とかになっているのかな。整数値で統一して実装しましょう。

雑な実装をしてお時間取らせてしまい本当に申し訳ありません、以後気を付けます。

mcz9mm commented 3 years ago

LGTM