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.

Sign up for GitHub

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

Jump to bottom

fix: SelectBirthBottomViewController 에서 피커뷰 변경 (#336) #343

Merged
merged 9 commits into from
Jan 31, 2023
Merged

fix: SelectBirthBottomViewController 에서 피커뷰 변경 (#336) #343

merged 9 commits into from
Jan 31, 2023

Conversation

hyun99999
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 피커뷰를 하나 더 추가해서 저번에 이야기했던 축 머시깽이가 좀 더 자연스러워 졌습니다.(아래의 모습에서 나아졌죠?)
    피커뷰가 좁게 존재하다보니 자연스러워졌습니다.

  • 회색 영역이 피커뷰의 맨 위에 올라와있기 때문에 글자가 좀 흐리게 보였던 거였어요.
  • 조금 길지만 채쌤과 함께 논의해서 제플린에 나와있는 것보다 둥글기를 수정하였습니다.(정적으로 23)
    https://nada-nrc8120.slack.com/archives/C02ASJ6BAMD/p1672907733679339
  • 완료 버튼을 누르게되면 앱이 종료되는데 기존에는 년도 포함해서 값을 보내주다가 지금은 월일만 보내주기 때문입니다. 이는 서버 데이터 모델이 나오면 명함 생성 뷰에서 받는 데이터를 수정하면서 같이 완성하겠심더. (fix: 명함생성뷰 생년월일 피커뷰에서 년도 삭제 #335 풀리퀘 때 fixme 주석으로 흔적은 남겨두었어용)

🚨참고사항

  • 그거 아셨나요? 👆피커뷰의 회색 영역은 첫 번째 subview 입니다. 이를 코드로 조정할 수 있었습니다.
    기존의 피커뷰에서의 글자 색이 흐렸던건 투명도가 0.8이었기 때문이죠!

스크린샷 2023-01-05 오후 5 27 58

- 여기 기본 회색 영역의 색입니다.

스크린샷 2023-01-05 오후 5 29 20

📸 스크린샷

기능 스크린샷
구현

📮 관련 이슈

@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Jan 5, 2023
@hyun99999 hyun99999 requested a review from dlwns33 January 5, 2023 09:41
@hyun99999 hyun99999 self-assigned this Jan 5, 2023
dlwns33
dlwns33 approved these changes Jan 31, 2023
View reviewed changes
Copy link
Member

@dlwns33 dlwns33 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 선생님 🙇‍♀️

NADA-iOS-forRelease/Sources/ViewControllers/BottomSheet/SelectBirthBottomViewController.swift
Comment on lines +127 to +130
let backgroundView = pickerView.subviews[1]
backgroundView.cornerRadius = 23
backgroundView.frame = CGRect(x: backgroundView.frame.minX, y: backgroundView.frame.minY, width: backgroundView.frame.width, height: 44)

Copy link
Member

Choose a reason for hiding this comment

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

풀리퀘 설명 읽었습니다! 요 부분이군용

Copy link
Member Author

Choose a reason for hiding this comment

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

맞슴다 신기허네요.. subview 로 다룰 수 있을줄은 몰랐는데여

@hyun99999 hyun99999 merged commit 9827e58 into develop Jan 31, 2023
@hyun99999 hyun99999 deleted the feature/#336 branch January 31, 2023 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlwns33 dlwns33 dlwns33 approved these changes

Assignees

@hyun99999 hyun99999

Labels
Fix 코드 수정 및 버그, 오류 해결 Hyungyu 🐯 현규 교수님 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix: SelectBirthBottomSheetViewController 에서 피커뷰 변경
2 participants
@hyun99999 @dlwns33