Closed yasyun closed 3 years ago
デザイン部分をもう少しfigmaに沿わせてからレビュー投げてください。
デザイン部分をもう少しfigmaに沿わせてからレビュー投げてください。
抜けていた部分を修正しました。申し訳ありませんでした。 僕の方では基本的に正しいデザインで描画されているのですが、もしkaitoさんの方でまだデザインが崩れているような場合があれば、使用しているシミュレーターの機種も教えていただけると幸いです。
すみません。レイアウトの問題ではなくそもそものfigmaの指定しているデザインと大きく異なっているのでその部分の修正ををお願いします。 (ユーザ名・ユーザIDの場所やNavigationBarのLargeTitleなど…)
すみません。レイアウトの問題ではなくそもそものfigmaの指定しているデザインと大きく異なっているのでその部分の修正ををお願いします。 (ユーザ名・ユーザIDの場所やNavigationBarのLargeTitleなど…)
レイアウト修正しました。しかし、Figmaの画面のお手本の画像と画面要件の方では、今回のようにデザインの面において非常に差異があると思います。最終課題を始めるにあたってどちらを参考にしてデザインを組み立てるかは、少なくとも僕が始めた時には明言されていなかったかと思うので、どちらのデザインを参考にするべきなのかはっきりと定めて課題を始める際に伝えた方が、より分かりやすくなるのではないかと感じました。
レビューをいただいた点の他に、ユーザーのアイコンが取得できなかった時に、エラー用の画像を表示するように修正しました。
レイアウト修正しました。しかし、Figmaの画面のお手本の画像と画面要件の方では、今回のようにデザインの面において非常に差異があると思います。最終課題を始めるにあたってどちらを参考にしてデザインを組み立てるかは、少なくとも僕が始めた時には明言されていなかったかと思うので、どちらのデザインを参考にするべきなのかはっきりと定めて課題を始める際に伝えた方が、より分かりやすくなるのではないかと感じました。
承知しました。 デザインカンプなのでそちらを参照してUIを作成していってください。
かなりfigmaのデザインに近づいていて良いと思います👍 もう少しデザインカンプに近づけるために、指定のマージンの値などに修正をお願いします。
ほぼほぼOKです! ・セルの文字列のはみ出し ・タイトルが長い場合は2行まで表示 上記2点が修正できたら僕からはLGTMです。
LGTMです!
今回、issueにしていたAlamofireの共通化をCommonApi.swiftというファイルに実装したので、そちらも合わせてレビューをお願いいたします。 issueはこちらです:Alamofire共通化