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

feat: 모바일 LNB Drawer #12

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

feat: 모바일 LNB Drawer #12

wants to merge 4 commits into from

Conversation

cs-shin
Copy link
Collaborator

@cs-shin cs-shin commented Feb 2, 2025

문제/이슈/주제: 모바일 LNB Drawer + 기타 수정

시안

스크린샷 2025-02-02 오후 4 16 24

상세

  • preview url :

개발

  1. / 로 진입 시 /chat으로 redirect
  2. 투표 페이지 추가 (/poll)
  3. system 테마 false 처리
  4. 모바일 LNB Drawer 기능
  5. 기타 리팩토링(폴더구조)

미비사항

Test

  • UI

    • PC view - chrome
    • PC view - firefox
    • PC view - safari
    • mobile view - chrome
    • mobile view - firefox
    • mobile view - safari
  • 기능

    • Test case 만들듯이 가능한 모든 경우의 수에 대해 체크박스 만들어 주세요

@cs-shin cs-shin self-assigned this Feb 2, 2025
Copy link
Collaborator

@youngminss youngminss left a comment

Choose a reason for hiding this comment

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

Drawer 잘 구현해줬네~ 👍👍👍
아래 사항만 확인해보면 좋을 것 같아 !

  1. dark mode 일 때 LNB 배경색
image
  1. icon 정렬 및 크기
image image
  1. LNB min-width 설정 필요해보임
image

@youngminss
Copy link
Collaborator

ResponsiveContainer 컴포넌트

눈치 못챘었는데 레이아웃 계산을 useEffect 에서 진행해서인지, 초기 렌더링은 Desktop 으로 설정된 후에 디바이스 크기에 맞는 레이아웃으로 재계산되어서 어색한 부분이 있는 것 같은데 확인해보면 좋을 것 같아 !

Feb-03-2025 15-11-08

@Najeong-Kim
Copy link
Collaborator

Najeong-Kim commented Feb 3, 2025

ResponsiveContainer 컴포넌트

눈치 못챘었는데 레이아웃 계산을 useEffect 에서 진행해서인지, 초기 렌더링은 Desktop 으로 설정된 후에 디바이스 크기에 맞는 레이아웃으로 재계산되어서 어색한 부분이 있는 것 같은데 확인해보면 좋을 것 같아 !

초기에 디바이스 크기로 렌더링하고 브라우저 크기로 업데이트를 해서 그런 것 같더라고! 그래서 휴대폰으로 볼 때는 디바이스 크기대로 보여서 괜찮던데 맞을까!??
애니메이션을 주면 덜 이상해보이려나

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