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

Refactor/#27: BottomSheet IOS height 오류 수정 및 각 커스텀 훅 init 로직 추가 #28

Merged
merged 8 commits into from
Mar 29, 2024

Conversation

semnil5202
Copy link
Member

@semnil5202 semnil5202 commented Mar 23, 2024

📄 Summary

close #27

BottomSheet IOS height 오류 수정 및 각 커스텀 훅 init(onReset) 로직 추가하여 반환값에 추가합니다.

IOS 환경에서 height: max-content 와 min-height, max-height 값을 같이 쓰면 height: max-content가 동작하지 않는 오류를 발견했습니다. 따라서 height 옵션을 제거하여 min-height와 max-height 사이간 height가 동적으로 변동하도록 합니다. (PC, IOS, Android 모두 정상 동작 확인 완료)

필터링 초기화 버튼에서 사용할 onReset 로직을 추가합니다. 기존에 BottomSheet 을 unmount 하는 방식에서 수동으로 값을 초기화 하도록 합니다. (Pop-Up 애니메이션 미동작 오류 대응)

🙋🏻 More

RadioContainer radio input wrap 시 상하 여백을 8px간 추가합니다.

@semnil5202 semnil5202 requested a review from yogjin March 23, 2024 13:58
@semnil5202 semnil5202 self-assigned this Mar 23, 2024
@semnil5202 semnil5202 added refactor 리팩토링 publish 버전 관련 labels Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
publish 버전 관련 refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BottomSheet IOS height 오류 수정 및 각 커스텀 훅 init 로직 추가
1 participant