Skip to content

Conversation

@l-lyun
Copy link
Contributor

@l-lyun l-lyun commented Jul 8, 2025

#️⃣연관된 이슈

📝작업 내용

키보드가 바텀 패딩은 잡고 있는데, 저희 고정 바텀버튼에 가려졌던 것으로 판단됩니다.
keyboardVerticalOffset이용하여 키보드가 올라갈때 화면이 얼마나 올라가는지 조정하여 해결된 것으로 판단됩니다.

스크린샷 (선택)

💬리뷰 요구사항(선택)

해당 속성 사용해 바텀버튼이 키보드 위로 항상 붙어오는데 피드백 부탁드립니다.

@l-lyun l-lyun requested a review from 99mini July 8, 2025 05:25
@l-lyun l-lyun self-assigned this Jul 8, 2025
Copy link
Member

@99mini 99mini left a comment

Choose a reason for hiding this comment

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

감사합니다 간단하게 해결할 수 있었네요!

<S.RegisterMarketContainer
behavior={Platform.OS === 'ios' ? 'padding' : 'height'}>
behavior={Platform.OS === 'ios' ? 'padding' : 'height'}
keyboardVerticalOffset={Platform.OS === 'ios' ? 90 : 0}>
Copy link
Member

Choose a reason for hiding this comment

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

90이라는 매직 넘부는 버튼의 텍스트 에어리어의 영역일까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

전자가 90px, 후자가 80px입니다
개발자도구 사용을 못해서 찍어보지 못했는데 버튼 위아래 여백 대칭이 90px이 맞는 것 같아서 진행했습니다

@99mini
Copy link
Member

99mini commented Jul 9, 2025

해당 속성 사용해 바텀버튼이 키보드 위로 항상 붙어오는데 피드백 부탁드립니다.

키보드에 항상 붙어있는데 disabled 인게 이상할 수 있어 보이네요. disabled condition을 이용하여 { condition && <Component /> } 이렇게 하는게 더 이뻐 보일려나

@l-lyun
Copy link
Contributor Author

l-lyun commented Jul 10, 2025

해당 속성 사용해 바텀버튼이 키보드 위로 항상 붙어오는데 피드백 부탁드립니다.

키보드에 항상 붙어있는데 disabled 인게 이상할 수 있어 보이네요. disabled condition을 이용하여 { condition && <Component /> } 이렇게 하는게 더 이뻐 보일려나

진행해볼까요 형님?

@99mini
Copy link
Member

99mini commented Jul 10, 2025

진행해볼까요 형님?

버튼이 아에 없으면 또 이 페이지에서 뭘하라는 건지 모를려나? 버튼이 없다가 나타나는 것과 disabled였다가 가능으로 변화는 것 중에 무엇이 더 좋을 지 모르겠네요. 일단 모르겠으니 현상 유지하는 걸로 하죠

@l-lyun
Copy link
Contributor Author

l-lyun commented Jul 10, 2025

진행해볼까요 형님?

버튼이 아에 없으면 또 이 페이지에서 뭘하라는 건지 모를려나? 버튼이 없다가 나타나는 것과 disabled였다가 가능으로 변화는 것 중에 무엇이 더 좋을 지 모르겠네요. 일단 모르겠으니 현상 유지하는 걸로 하죠

그러면 브랜치 지금상태에서 커밋 더안달고 머지하겠습니다!!
머지하고 바로 테스트플라이트 서버 개발환경으로 올려놓겠습니다.

@l-lyun l-lyun merged commit 8452be6 into main Jul 10, 2025
1 check passed
@l-lyun l-lyun deleted the fix/register-screen-keyboard-padding branch July 10, 2025 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants