[Fix] 바텀시트 내부 버튼 폭 수정 및 컨텐츠 간격 조정 #235
Open
+33
−13
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#️⃣ Related Issue
Closes #231
✅ Done Task
[x] 바텀시트 내부 버튼 사이즈 폭 조정
[x] 컨텐츠 간격 조정
☀️ New-insight
다양한 사이즈에 대응하기 위해 구체적인 사이즈를 지정하는 것을 지양해야겠다는 생각이 들었어요!
💎 PR Point
1️⃣ 공통 컴포넌트(Toggle) 폭 수정
: 해당 컴포넌트는 지은, 예림 뷰에서만 사용되는 공통 컴포넌트입니다.
지은 뷰에서 폭 100%로 수정하여 피그마 뷰와 일치 시켰고, 제 컴포넌트에서 별도의 레이아웃을 추가하여 조정 완료 했습니다!
2️⃣ 바텀시트 하단 버튼 폭 수정
:
width: 27.4rem;
구체적인 사이즈를 지정한 기존의 코드를 삭제하고
아래의 내용으로 수정하여 피그마 뷰와 일치 시켰습니다!
flex-grow: 1;
3️⃣ 바텀시트 내부 컨텐츠 간격 조정
: 기존 1rem을 피그마 뷰와 동일하게 1.5rem으로 수정하였습니다.
🍀 추가적으로 공통 토글 컴포넌트는 지은이 컴포넌트라 코리 페어는 아니지만 지은이도 코드리뷰어로 소환하였습니다!
지은이는 1️⃣번의 내용만 확인해주시면 될 거 같습니당!
🍀 브랜치명이 적합하지 않다고 생각되어 머지할때 수정하겠습니다!
📸 Screenshot
1️⃣,2️⃣,3️⃣번의 내용 모두 아래의 사진으로 확인해주시면 감사하겠습니다!
2024-09-20.1.51.43.mov