Merged
Conversation
Pr 템플릿 추가 및 Main 폴더 구조화 완료
헤더, 메인페이지 네비게이션 분리
karpitony
approved these changes
Feb 14, 2025
Member
There was a problem hiding this comment.
작업해 주신 부분 잘 봤습니다, 수고하셨습니다 👍👍
보면서 크게 문제되는 부분은 아니지만, 몇 가지 의견이 있어 공유드립니다.
1. 목차 헤더가 최상단에서도 유지되면 좋을 것 같습니다.
- 랜딩 페이지의 목차는 자주 확인하게 되는데, 상단에서 나타났다 사라졌다를 반복하니 신경이 쓰입니다.
- 최상단에서도 유지되면, 지원 요건 등 원하는 페이지로 바로 이동할 수 있어 UX가 더 좋아질 것 같습니다.
2. 트랙 소개와 인트로 페이지 사이 간격이 좁아졌습니다.
| 현재 PR | 기존 배포 |
|---|---|
 |
 |
현재 PR 브랜치와 기존 배포본을 비교해보니, 트랙 소개와 인트로 사이의 간격이 약간 좁아진 것 같습니다.
큰 차이는 아니지만, 간격을 조금만 넓히면 더 깔끔하고 시원한 느낌을 줄 것 같아요!
3. 오픈 예정 모달에 오타가 있습니다.
- 아마 vscode에서 한글 입력하면 생기는 버그가 그대로 들어간 것 같습니다.
- 이 부분은 밑에 깃허브 conversation 걸어놓겠습니다!
3번 부분만 수정하시고 머지해도 될 것 같습니다.
1번과 2번은 한 번 고민해보시고, 괜찮다고 생각되시면 반영해 주시면 좋을 것 같습니다. 😊
| isOpen={isPopupOpen} | ||
| onClose={() => setPopupOpen(false)} | ||
| title={"아직 오픈되지 않았습니다."} | ||
| content={"오픈픈 예정: 2025년 4월"} |
Contributor
Author
There was a problem hiding this comment.
아 넵 고치겠습니당
Member
|
깃 브랜치 전략을 feature -> dev -> main으로 하기로 했어서 머지 대상 브랜치를 dev로 바꿔놓았습니다. |
dewbeeny
reviewed
Feb 15, 2025
Contributor
dewbeeny
left a comment
dewbeeny
left a comment
There was a problem hiding this comment.
다 확인했고 윤석님이 말한 부분 위주로 조금만 더 수정해주면 될 것 같아요! 하나만 덧붙이자면 목차 헤더 상단에 margin 여백을 더 두는게 어떨까요?! 너무 수고하셨습니다!! 수정해서 다시 올려주시면 머지할게요!
dewbeeny
added a commit
that referenced
this pull request
Feb 15, 2025
Feature/#20 메인페이지 반응형 작업(트랙 소개 제외)
4 tasks
karpitony
pushed a commit
that referenced
this pull request
Mar 12, 2025
feat: apiConfig.ts에 인터셉터 추가하여 엑세스 토큰이 만료되었을 때(401) 토큰 재발급 기능 추가
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
#️⃣연관된 이슈
📝작업 내용
✅ 체크리스트
스크린샷 (선택)
bandicam.2025-02-15.00-32-18-063.mp4