[Fix/#436] 사이드바 하단 잘리는 오류 수정 - 2 #442
Merged
+18
−1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶 |
|
sinji2102
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
📌 관련 이슈번호
🎟️ PR 유형
어떤 변경 사항이 있나요?
✅ Key Changes
📢 To Reviewers
#441 PR에서 실패해서 같은 이슈/브랜치에서 작업했습니다! 자세한 내용은 해당 PR에서 확인하면 될 것 같아요!
제 생각엔... 처음 높이를 계산하고 이후에 계산을 안 해서 높이가 바뀔 때마다 리렌더링이 필요한 것 같아 높이가 바뀔 때마다 이너 뷰포트 크기를 계산했습니다!
높이가 바뀔 때마다 dvh를 계산하는 방법도 고민해보았지만, vw, vh 단위는 브라우저의 스크롤 영역까지를 포함시킨 뷰포트 크기를 반환하여 모바일에서 사용하기 좋은 방식이 아니라고 해
innerHeight를 선택하였습니다!처음에 참고한 아티클은 [css 단위] dvh, svh, lvh - 모바일 주소 표시줄 영역 대응인데, 왜 dvh가 안 먹히는지 잘 모르겠습니다... 😢 분명 dvh는 유동적으로 대응한다구 했는데!! 혹시 이유를 알게 된다면... 코드 리뷰 남겨 주세요... 🫶 모른다면 무시해두 댑니다 급한 스프린트 쳐내구 제가 찾아볼게요!!!!!!!
모바일로 빌드해야만 확인할 수 있어서 코드리뷰 없이 바로 머지하겠습니다!!
📸 스크린샷
20분째 시도하고 있는데 용량 커서 안 올라가네요... 신지바보에서 확인하세요...........
🔗 참고 자료
[React] 반응형으로 항상 화면에 꽉 찬 컴포넌트 만들기