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] 바텀시트 내부 버튼 폭 수정 및 컨텐츠 간격 조정 #235

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

yarimu
Copy link
Contributor

@yarimu yarimu commented Sep 16, 2024

#️⃣ 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-17 오전 12 23 48
  • 모바일뷰 풀었을때
스크린샷 2024-09-17 오전 1 33 56
  • 바텀시트 내부 핸들바 중앙정렬 사이즈별 동영상
2024-09-20.1.51.43.mov

@yarimu yarimu added 🛠 Fix 기존의 버그 수정 예림 labels Sep 16, 2024
@yarimu yarimu self-assigned this Sep 16, 2024
Copy link
Member

@j-nary j-nary left a comment

Choose a reason for hiding this comment

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

야무진 야리무 LGTM 🚀
상세한 피알 덕분에 코리 시간이 절반으로 줄어드네용 섬세한 배려에 감사드립니당 💓

@yarimu
Copy link
Contributor Author

yarimu commented Sep 19, 2024

웁스바리 바텀시트 내부 핸들바 가운데 정렬 하여 추가로 푸시 날렸습니다! pr 전에 한번더 확인해보겠습니당! 해당 내용은 가장 마지막 동영상 확인해주세요! 감사합니도!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 기존의 버그 수정 size/s 예림
Projects
Status: 🚨 Queued
Development

Successfully merging this pull request may close these issues.

[ Fix ] 바텀시트 내 버튼 크기 조정 및 컨텐츠 간격 조정
2 participants