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

[#185] 15+의 모달 Sheet UI 작업 완료 #186

Merged
merged 5 commits into from
Jan 3, 2025
Merged

Conversation

CJiu01
Copy link
Collaborator

@CJiu01 CJiu01 commented Nov 28, 2024

🍎 EATSSU iOS Team Pull Request

🔆 개요

  • 15+의 모달 Sheet UI 작업 완료하였습니다.

💡 PR Point

  • HomeVC의 rightBarButtonTapped 함수는 test하기위한 임시 코드입니다. (해당 부분 제외하고 봐주세요)

📷 스크린샷

스크린샷 2024-11-28 오후 6 41 07 image

📮 관련 이슈

ios16+ 에서 지원하는 UI입니다.
로그인이 필요한 서비스예요. 3초만에 로그인하고 기능을 사용해보세요!
@CJiu01 CJiu01 self-assigned this Nov 28, 2024
@HI-JIN2
Copy link
Member

HI-JIN2 commented Dec 31, 2024

스크린샷 2024-12-31 오후 12 46 55

세로 길이가 너무 큰 것 같지 않나요? 15+이면 길이 커스텀이 된다고 했던 것 같은데, 피그마랑 비율 차이가 너무 나는 것 같아요~ @CJiu01

@HI-JIN2 HI-JIN2 requested a review from jayn2u December 31, 2024 03:58
@jayn2u jayn2u requested a review from HI-JIN2 January 1, 2025 13:27
@jayn2u
Copy link
Member

jayn2u commented Jan 1, 2025

일단 중간에 로고가 있고 없고의 차이네요. 이 부분은 조정해서 맞추면 될 듯 싶습니다.

@CJiu01
Copy link
Collaborator Author

CJiu01 commented Jan 2, 2025

첨부해주신 디자인(로고 없는 버전)은 아래 pr에 작업해두었습니다.
커스텀은 16+부터 가능하여 이하 버전일 시에 로고있는 버전으로 대응됩니다.

#184

@jayn2u
Copy link
Member

jayn2u commented Jan 2, 2025

해당 PR이 계속 열려 있는 이유에 대해서 알 수 있을까요?

@CJiu01
Copy link
Collaborator Author

CJiu01 commented Jan 3, 2025

해당 PR이 계속 열려 있는 이유에 대해서 알 수 있을까요?

approved 확인 후 닫으려했습니다. 확인됐으니, 해당 pr은 병합하겠습니다~

@CJiu01 CJiu01 merged commit 444c20f into fix/loginModalSheet Jan 3, 2025
@CJiu01 CJiu01 deleted the feat/#185 branch January 3, 2025 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants