-
Notifications
You must be signed in to change notification settings - Fork 1
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: 헤더 컴포넌트 UI 추가 #19
Merged
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
media.tsx 파일에 반응형 크기 설정 파일 생성
PC 화면과 태블릿/스마트폰 화면에서 헤더 디자인이 많이 달라서, PC 헤더를 ContainerLarge 컴포넌트로 분리 정의된 브레이크 포인트에 따라 화면 나눔
태블릿/휴대폰 화면에 있는 둥근 아이콘 메뉴를 HeaderIconMenu 컴포넌트로 분리함.
develop 브랜치에서 로그인 모달 내용 병합함
크기에 따라 헤더 내 아이콘 등의 위치 수정하여 반응형 헤더 UI 개발
피그마 디자인에 맞추어 small 화면 기준 390px로 수정
화면 크기가 media.small보다 크고 553px보다 작을 때 장마당 로고 이미지 위치 고정
헤더에 래플 업로드 버튼 추가
Header 파일에서 사용하지 않는 이미지 import 삭제
모달 관련 업데이트된 develop 브랜치 pull 받아오고 충돌 해결
래플 업로드 버튼 눌렀을 때 래플 업로드 페이지(/raffle-upload)로 이동하도록 연결
헤더 상단에 위치한 로그인/알림/설정 메뉴 위치 수정 헤더 하단 구분선을 border-bottom 속성에서 없애고 바깥에 div 요소로 구분
헤더 전체에 z-index:100 부여하여 카테고리 메뉴 z-index 올리기 시도 홈 페이지와 병합했을 때 의도한 것과 같이 카테고리 메뉴가 상단에 올라오는지 확인 필요
- 아이콘 이미지 및 글자 크기 수정 - 카테고리 메뉴 호버 시 테두리 생기도록 이벤트 추가
Header 파일에서 사용하지 않는 styled component 및 주석 삭제 close issue #1
jade653
approved these changes
Jan 28, 2025
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.
이슈 번호
#1
작업 내용
헤더 컴포넌트 반응형 추가 및 디테일 추가
변경 사항
남은 이슈