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 ] 모바일뷰 캘린더 하단바 안가려지게 수정 #335

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

ijieun
Copy link
Collaborator

@ijieun ijieun commented Oct 31, 2024

#️⃣ Related Issue

Closes #334

✅ Done Task

  • 모바일뷰 캘린더 하단바 안가려지게 수정

☀️ New-insight

💎 PR Point

크롬과 사파리 모두 특정 버전 이후부터는 스크롤 시 주소창과 하단 바가 자동으로 숨겨지는 기능을 가지고 있었습니다.!
모바일 웹 스크롤 시 하단바에 가려지던 바텀바가 가려지지 않도록 수정했습니다.
height : 100vh 대신 height : auto 와 min-height: 100dvh를 사용해서,
전체화면으로 스크롤되도록 하여 해결했습니다!

📸 Screenshot

모바일뷰 테스트

  • 하단바 있을 경우
    IMG_2084

  • 하단바 없을 경우
    IMG_2085

@ijieun ijieun added 🛠 Fix 기존의 버그 수정 지은 labels Oct 31, 2024
@ijieun ijieun self-assigned this Oct 31, 2024
@ijieun ijieun requested review from lydiacho and se0jinYoon October 31, 2024 15:33
@lydiacho lydiacho changed the title fix: 모바일뷰 캘린더 하단바 안가려지게 수정 [ Fix ] 모바일뷰 캘린더 하단바 안가려지게 수정 Nov 1, 2024
Copy link
Collaborator

@lydiacho lydiacho left a comment

Choose a reason for hiding this comment

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

좋아용
혹시 그냥 height: 100dvh 로 바꾸는거랑 지금처럼 auto + min-height 해주는거랑 무슨 차이가 있나요??

Copy link
Collaborator

@se0jinYoon se0jinYoon left a comment

Choose a reason for hiding this comment

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

좋슴니다!
근데 이렇게 스크롤에 따라 바뀔 수 있는 뷰는 화면 녹화로 올려주면 좋을 것 같아요!

@ijieun
Copy link
Collaborator Author

ijieun commented Nov 3, 2024

@lydiacho
height: 100dvh는 요소가 항상 뷰포트 전체 높이에 고정되지만 height: auto; min-height: 100dvh는 기본적으로 뷰포트 높이를 차지하면서도 콘텐츠가 많아지면 요소의 높이가 자동으로 확장됩니다. 이렇게 하면 내부에 추가 콘텐츠가 있을 때 height: auto 설정으로 인해 전체 화면에서 자연스럽게 스크롤이 가능해집니다.

근데.. 높이가 뷰포트에 고정되어 있기 때문에 유동적인 높이 조절 기능은 필요하지 않겠다는 생각이 드네요 🤔 빼줘야겠습니다

@ijieun ijieun merged commit 24d5a32 into develop Nov 4, 2024
1 check passed
@ijieun ijieun deleted the fix/#334/showBottomsheetBar branch November 4, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 기존의 버그 수정 size/xs 지은
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Fix ] 휴대폰에서 캘린더 바텀시트 하단바 가려지는 이슈 해결
3 participants