Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/upload page #61

Merged
merged 10 commits into from
Jan 16, 2020
Merged

Feat/upload page #61

merged 10 commits into from
Jan 16, 2020

Conversation

MasatakaKudou
Copy link

関連Issue

このPRと関連するIssueの番号を書きます
アップロード画面
close #24

やったこと

やったことを書きます
アップロード画面の実装、遷移(ホームからアップロード、アップロードからホーム)、ホーム画面にアップロード画面に遷移するボタン実装

実装の詳細

どのように実装したかを書きます(なるべく詳しく書いた方が良い)
アップロード画面の実装(iphone5s以外はレイアウトが崩れないようにできました。)
遷移(値の受け渡しは一つもしないでただ遷移してるだけ)
ホーム画面にボタン実装(右下に固定したボタンを作成)

レビューして欲しいところ

コード汚いですけど、写真で判断してください。

スクリーンショット(あれば)

UIに変更を加えた祭はスクショがあるとわかりやすいです。変更前・変更後のスクショがあると良い
iPhone X

iPhoneX

iPhone 6/7/8

iPhone6:7:8

Pixel 2

Pixel2

PC

PC

ホーム画面に実装したボタン

finds_home

@MasatakaKudou MasatakaKudou requested a review from b4tchkn January 15, 2020 23:54
@MasatakaKudou MasatakaKudou self-assigned this Jan 15, 2020
@b4tchkn b4tchkn merged commit 3591afc into develop Jan 16, 2020
@b4tchkn b4tchkn deleted the feat/uploadPage branch January 16, 2020 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants