Skip to content

feat: 인트로, 헤더 레이아웃 1.5차 완성#16

Merged
dewbeeny merged 7 commits intodevelopfrom
Feature/12-create-intro-header-section
Feb 13, 2025
Merged

feat: 인트로, 헤더 레이아웃 1.5차 완성#16
dewbeeny merged 7 commits intodevelopfrom
Feature/12-create-intro-header-section

Conversation

@dongmin0204
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

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

Choose a reason for hiding this comment

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

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

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
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
karpitony pushed a commit that referenced this pull request Mar 12, 2025
fix: TypeScript 타입 오류 수정 및 API 응답 타입 개선
karpitony pushed a commit that referenced this pull request Mar 12, 2025
fix: 환경별 리다이렉트 url 변경
karpitony pushed a commit that referenced this pull request Mar 12, 2025
fix: 리디렉트 url 엔드포인트 변경
karpitony pushed a commit that referenced this pull request Mar 12, 2025
fix: Vercel 라우팅 문제 해결을 위한 vercel.json 추가
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