shinonome-inc / mobile_tak848

0 stars 0 forks source link

【最終課題】 #11 ログイン画面の作成 #12

Closed TAK848 closed 2 years ago

TAK848 commented 2 years ago

Issue

resolved #11

最終課題項目

PR出すまで勘違いしたいたのですが,本来ログイン画面だけで出すべきでした。大変申し訳ございません。分割するのが 困難なためこのまま提出させていただきます。

追加内容概要

Localizationも要件にはないので流石にPR分けるべきでした,すみません…

フォルダについて

動作gif

TAK848 commented 2 years ago

FigmaのPagesに,動画の解説にあったIconsがなかったため,過去の履歴を遡りduplicateしてダウンロードしました。 ついでにそこから色の定義を引っ張ってAssetに定義しました。

Kaito-a-bit commented 2 years ago
スクリーンショット 2022-02-15 9 50 29

ログインせずに利用するボタンについているこの制約は必要ないかと思います。

TAK848 commented 2 years ago

が、LocalizationについてのExtensionをまとめたものとして一つのSwiftファイルに書いてしまっても良いのではと思いました。ここについては何か拓也さんの意図があれば伺いたいです。

意図は,ここをもとに,各クラスのExtensionとしての見通しを良くすることでした。 ですが現状数が少ないですし,やっていることは実質同じ機能なのに冗長にフォルダが分かれているようにも感じられるので,Utilityフォルダにまとめることにしました。ありがとうございます!該当commit Extensionを大量に使って見通しが悪くなりそうであれば再度Extensionフォルダを作ってみることを検討したいと思います!

スクリーンショット 2022-02-15 9 50 29

ログインせずに利用するボタンについているこの制約は必要ないかと思います。

完全に盲点でした!ありがとうございます!!修正しました