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

[FIX] 온보딩 UI #219

Merged
merged 2 commits into from
Mar 17, 2023
Merged

[FIX] 온보딩 UI #219

merged 2 commits into from
Mar 17, 2023

Conversation

WhiteHyun
Copy link
Member

📌 PR 요약

🌱 작업한 내용

  • 수정된 Lottie Image를 적용헀습니다.
  • Lottie Image 사이즈를 1:1.36 비율로 설정하였습니다.(328/240 = 1.3666...)

📸 스크린샷

스크린샷

📮 관련 이슈

디자인 파트에서 UI수정이 이루어졌고 그에 맞게 적용
특히 Lottie Image는 정적 사이즈가 아니라 width에 맞게 height를
비율로 설정하여 모든 아이폰에 같은 비율의 Image가 보이도록 설정
@WhiteHyun WhiteHyun added 🍎 ConfigureUI 화면 프로토타입 UI 구현 D+4~ 그다지 중요하지 않는 사소한 변경이에요. 편할 때 PR 남겨주세요 📕Onboarding labels Mar 16, 2023
@WhiteHyun WhiteHyun self-assigned this Mar 16, 2023
Copy link
Contributor

@soobin-k soobin-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 승현님!

@@ -51,7 +51,8 @@ final class OnboardingViewController: BaseViewController {
override func setupConstraints() {
super.setupConstraints()
onboardingView.snp.makeConstraints {
$0.size.equalTo(240)
$0.directionalHorizontalEdges.equalToSuperview().inset(16)
$0.height.equalTo(onboardingView.snp.width).dividedBy(1.36)
$0.centerX.equalToSuperview()
$0.centerY.equalToSuperview().dividedBy(1.564)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

snapkit 내부에 dividedBy 이런게 있었군요 꿀정보 감사합니다~!👍👍

Copy link
Contributor

@dlrjswns dlrjswns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

엇 저 부분 수정하셨군요 수고하셨습니다 승현님 ✔️

@WhiteHyun WhiteHyun merged commit 1dfcb81 into develop Mar 17, 2023
@WhiteHyun WhiteHyun deleted the feat/138-Onboarding/UI branch March 17, 2023 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍎 ConfigureUI 화면 프로토타입 UI 구현 D+4~ 그다지 중요하지 않는 사소한 변경이에요. 편할 때 PR 남겨주세요 📕Onboarding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants