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

[#159] Header 관련 로직 추가 #160

Merged
merged 14 commits into from
Jan 4, 2024
Merged

[#159] Header 관련 로직 추가 #160

merged 14 commits into from
Jan 4, 2024

Conversation

halfmoon-mind
Copy link
Member

Issue

Description

얘기할 것

  • 메인화면이 반쯤 잘려서 보여요... 이거 어떻게 하죠??

수정사항

  • Header의 로고 터치 시, /로 이동
  • 알림 터치 시, 현재 관련 뷰가 구현 안되어 있어서 주석 처리
  • nav에서 notice로 연결되는 라우팅이 기존에 /announce로 되어 있던 부분 수정
  • 조건에 따라서 헤더 결정 추가

Check List

  • PR 제목을 커밋 규칙에 맞게 작성
  • PR에 해당되는 Issue를 연결 완료
  • 적절한 라벨 설정
  • 작업한 사람 모두를 Assign
  • 작업한 팀에게 Code Review 요청 (Reviewer 등록)
  • main 브랜치의 최신 상태를 반영하고 있는지 확인

Screenshot

스크린샷 2023-12-28 오후 10 54 26

@halfmoon-mind halfmoon-mind added the feat 🔧 새로운 기능 관련된 Issue label Dec 28, 2023
@halfmoon-mind halfmoon-mind self-assigned this Dec 28, 2023
Copy link

vercel bot commented Dec 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
daitssu-client ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 2, 2024 11:43am

Copy link
Member

@SujinKim1127 SujinKim1127 left a comment

Choose a reason for hiding this comment

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

배포 쪽에서 오류가 나는것 같아요!
확인부탁드립니당~!

@halfmoon-mind
Copy link
Member Author

수정했어용

@SujinKim1127 SujinKim1127 self-requested a review January 4, 2024 12:52
Copy link
Member

@SujinKim1127 SujinKim1127 left a comment

Choose a reason for hiding this comment

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

고생하셨습니당!!!

@halfmoon-mind halfmoon-mind merged commit 7be564c into main Jan 4, 2024
2 checks passed
@halfmoon-mind halfmoon-mind deleted the feat/#159 branch January 4, 2024 12:59
SujinKim1127 added a commit that referenced this pull request Jan 26, 2024
* feat: 상세정보 api 연결

* feat: 좋아요, 스크랩 api 연결

* feat: 댓글 api 연결

* style: 반응형으로 변경

* style: 배경 흰색으로 변경

* [#159] Header 관련 로직 추가 (#160)

* feat: Route에 따른 header 조건 지정

* fix: 바텀네비게이션 announce -> notice로 수정

* refactor: style 코드들 정렬

* feat: 더보기 클릭 라우팅 추가

* feat: 코드 스타일 업데이트

* feat: 스크롤 코드 삭제

* feat: bottom nav bar true로 설정

* feat: 상위 3개만 출력하도록 변경

* feat: 로그인에서 utility header 추가

* refactor: 불필요한 코드 삭제

* refactor: PR시 빌드 확인 코드 삭제

* feat: main에서 api 콜 수정

* refactor: header 설정 추가

* fix: storybook 업데이트

* style: 빠진 글자 수정

* [#164] 레이아웃 관련 스타일 수정 (#165)

* style: global.css body 요소에 height 설정, padding 추가(네비게이션바 100px)

* style: 랜딩 페이지 배경 이미지 position:fixed로 변경

* fix: 랜딩 페이지에서 헤더 미노출 로직 추가

* feat: 상세정보 api 연결

* feat: 좋아요, 스크랩 api 연결

* feat: 댓글 api 연결

* style: 반응형으로 변경

* style: 배경 흰색으로 변경

* style: 빠진 글자 수정

* chore: console 삭제

* feat: 댓글에 닉네임 추가

* style: 댓글 디자인 반응형으로 수정

---------

Co-authored-by: Sanghyeon Sim <[email protected]>
Co-authored-by: 김정우 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 🔧 새로운 기능 관련된 Issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 헤더 로고 터치시 홈으로 돌아가기, 스크롤해서 preview 제거
2 participants