홈페이지 sidebar overflow되는 문제 #232
Merged
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
eugene028
reviewed
Nov 29, 2023
| className={ | ||
| 'relative bg-white-100 w-full h-[750px] overflow-x-hidden overflow-y-scroll' | ||
| }> | ||
| <div className={'relative bg-white-100 h-[750px] overflow-hidden'}> |
There was a problem hiding this comment.
overflow-hideen 으로 스크롤되는 문제 해결하셨군요!? 확인했습니다!
혹시 높이는 750px로 고정한 이유가 있을까용?!
There was a problem hiding this comment.
지금 레이아웃을 보시면
header
homePage
navbar
이렇게 3등분이 돼 있는데 setting을 넣기 위해서 최대한 homepage전체 높이를 고정시켜놓는 방법을 선택했습니다.
homepage가 더 커져버리면 scroll이 계속 되더라구요.. div로 감싸면 또 따로 animation을 또 적용시켜줘야해서요!
그래서 header와 navbar를 최대한 방해하지 않는 선에서 750px을 선택했습니다.
There was a problem hiding this comment.
옹 그렇군요!! 그런데 화면이 매우 작은 반응형 화면에서 볼때는 Nav바가 가려지게 되는 것 같아서 전체 vh 높이에서 nav바랑 header만큼의 길이를 계산해서 빼는 방식으로 구현해도 좋을 거 같다는 생각이 들었습니다!
jisung24
added a commit
that referenced
this pull request
Nov 29, 2023
* fix: 마이페이지 로컬주소 해결 (#206) * fix: 마운트시 나타나는 데이터 제거 (#224) * feat: silder 바 제작 * likeAcademy 버그 수정 (#229) * fix: 마운트시 나타나는 데이터 제거 * fix: get요청 확인 * fix: 머지 미반영 코드 추가 * fix: likeAcademy page 수정 * fix: 안 쓰는 변수 제거 * fix: scroll 없애고, 전체표시 * fix: 아코디언 가운데정렬, 메모 스크롤 적용 * fix: 불필요한 border제거 * hotfix: 바텀시트 수정 * hotfix: 좋아요 api 추가 및 ReviewPercent 타입 변경 * hotfix: 맵 페이지 검색바 로직 분리 및 지도가 보이지 않는 문제 수정 * hotfix: 스토리북 에러 수정 * hotfix: 검색결과 클릭시 이동 후 마커 표시되도록 수정 * feat: toast ui 생성 (#230) * hotfix: 필터 페이지 수정 * hotfix: 바텀시트 스토리북 삭제 * hotfix: 코드리뷰 반영 * fix: schedule detail api get요청 수정 반영 * hotfix: 코드리뷰 반영 * fix: 안 쓰는 변수 삭제 * fix: delete api수정 * fix: homepage settingpage잘 작동 (#232) --------- Co-authored-by: Eugene Kim <[email protected]> Co-authored-by: sincerity developer <[email protected]> Co-authored-by: Eugene Kim <[email protected]>
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.
내용 설명
홈페이지에 대해, 버튼의 위치, settingpage overflow방지를 작업했습니다
구현 내용
스크린샷?
overflow 방지
참고 사항
궁금한 점
close #231