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: 인트로, 헤더 레이아웃 1.5차 완성 #16

Merged
merged 7 commits into from
Feb 13, 2025

Conversation

dongmin0204
Copy link
Contributor

@dongmin0204 dongmin0204 commented Feb 13, 2025

#️⃣연관된 이슈

#12

📝작업 내용

마우스 호버 추가.
디자인 및 레이아웃
팝업 추가 -> 이미 수빈님이 하신 거로 통일 했습니다.

✅ 체크리스트

  • 기능이 정상적으로 동작하는지 확인
  • 코드 리뷰 피드백 반영
  • UI/UX 디자인 적용 확인

스크린샷 (선택)

스크린샷 2025-02-13 오전 9 59 50

2025-02-13.10.00.01.mov

@dongmin0204
Copy link
Contributor Author

아 지금 종속성 관련 파일 잘못 건든 것 같습니다.

</S.TextBlock>
<S.Nav>
<S.NavList>
<li><a href="#home">홈</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

image
헤더 라우팅이 잘못된 것 같습니다...
저희 헤더가 두개가 있는데 디자인 해주신건 최상단 헤더같은데 초기 세팅해주신 헤더는 두번째 헤더(목차)인것 같습니다.

이부분 고쳐주시면 머지해도 될 것 같습니다!!
추후 수빈님 PR까지 머지하면 팝업을 공통 컴포넌트로 분리하는 방안도 좋아보입니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 지금 일정으로 인해 밖이라, 오늘 밤까지 해결하겠습니다! 감사합니다:)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 지금 일정으로 인해 밖이라, 오늘 밤까지 해결하겠습니다! 감사합니다:)

@dewbeeny dewbeeny merged commit 04c9bd7 into develop Feb 13, 2025
1 check passed
@karpitony karpitony deleted the Feature/12-create-intro-header-section branch February 15, 2025 15:12
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