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

Feature/#19 main page intergrate #20

Merged
merged 6 commits into from
Feb 15, 2025
Merged

Conversation

parkjoohyung0826
Copy link
Contributor

@parkjoohyung0826 parkjoohyung0826 commented Feb 14, 2025

#️⃣연관된 이슈

closed #19

📝작업 내용

반응형 제외 메인페이지 통합

✅ 체크리스트

  • 모달 컴포넌트화
  • 유니온 클릭 시 이동 기능 추가
  • 백그라운드 블러 효과 적용
  • 헤더 클릭 시 모달창
  • 메인페이지 네비게이션
  • 트랙소개 부분 가독성 수정

스크린샷 (선택)

bandicam.2025-02-15.00-32-18-063.mp4

Copy link
Contributor

@karpitony karpitony left a comment

Choose a reason for hiding this comment

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

작업해 주신 부분 잘 봤습니다, 수고하셨습니다 👍👍

보면서 크게 문제되는 부분은 아니지만, 몇 가지 의견이 있어 공유드립니다.

1. 목차 헤더가 최상단에서도 유지되면 좋을 것 같습니다.

  • 랜딩 페이지의 목차는 자주 확인하게 되는데, 상단에서 나타났다 사라졌다를 반복하니 신경이 쓰입니다.
  • 최상단에서도 유지되면, 지원 요건 등 원하는 페이지로 바로 이동할 수 있어 UX가 더 좋아질 것 같습니다.

2. 트랙 소개와 인트로 페이지 사이 간격이 좁아졌습니다.

현재 PR 기존 배포
image image

현재 PR 브랜치와 기존 배포본을 비교해보니, 트랙 소개와 인트로 사이의 간격이 약간 좁아진 것 같습니다.
큰 차이는 아니지만, 간격을 조금만 넓히면 더 깔끔하고 시원한 느낌을 줄 것 같아요!

3. 오픈 예정 모달에 오타가 있습니다.

  • 아마 vscode에서 한글 입력하면 생기는 버그가 그대로 들어간 것 같습니다.
  • 이 부분은 밑에 깃허브 conversation 걸어놓겠습니다!

3번 부분만 수정하시고 머지해도 될 것 같습니다.
1번과 2번은 한 번 고민해보시고, 괜찮다고 생각되시면 반영해 주시면 좋을 것 같습니다. 😊

isOpen={isPopupOpen}
onClose={() => setPopupOpen(false)}
title={"아직 오픈되지 않았습니다."}
content={"오픈픈 예정: 2025년 4월"}
Copy link
Contributor

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.

아 넵 고치겠습니당

@karpitony karpitony changed the base branch from main to develop February 14, 2025 17:47
@karpitony
Copy link
Contributor

깃 브랜치 전략을 feature -> dev -> main으로 하기로 했어서 머지 대상 브랜치를 dev로 바꿔놓았습니다.
혹시 main으로 머지를 의도하신거라면 다시 바꿔주세요! 🙇‍♂️

Copy link
Contributor

@dewbeeny dewbeeny left a comment

Choose a reason for hiding this comment

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

다 확인했고 윤석님이 말한 부분 위주로 조금만 더 수정해주면 될 것 같아요! 하나만 덧붙이자면 목차 헤더 상단에 margin 여백을 더 두는게 어떨까요?! 너무 수고하셨습니다!! 수정해서 다시 올려주시면 머지할게요!

dewbeeny added a commit that referenced this pull request Feb 15, 2025
Feature/#20 메인페이지 반응형 작업(트랙 소개 제외)
@dewbeeny dewbeeny merged commit eb945a2 into develop Feb 15, 2025
1 check passed
@karpitony karpitony deleted the Feature/#19-MainPage-Intergrate 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.

[Feat] 메인페이지 통합 및 기타 기능 수정(반응형 제외)
3 participants