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 - Header #39

Merged
merged 4 commits into from
May 4, 2024
Merged

✨ Feature - Header #39

merged 4 commits into from
May 4, 2024

Conversation

shimseohyun
Copy link
Contributor

@shimseohyun shimseohyun commented May 1, 2024

PR 타입 ( 하나 이상의 PR 타입을 선택해주세요 )


  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

개요


해더, 모바일 스와이퍼 구현 스크린샷 2024-05-01 오후 9 13 05

변경 사항


코드 리뷰시 참고 사항


이미지의 경우 추후 작업해서 마무리 하겠습니다.
stories 부분 스와이퍼가 두개 뜨고 있는데 이것도 1차 피드백 이후 애니메이션 형태에 맞추어 수정하며 수정하겠습니다

@shimseohyun shimseohyun self-assigned this May 1, 2024
@shimseohyun shimseohyun linked an issue May 1, 2024 that may be closed by this pull request
4 tasks
Copy link
Member

@puretension puretension left a comment

Choose a reason for hiding this comment

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

수고많으셨습니다!!

Copy link
Contributor

@bunju20 bunju20 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Contributor

@bianbbc87 bianbbc87 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

Copy link
Contributor

@naarang naarang left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!☺️

Copy link
Member

@seochan99 seochan99 left a comment

Choose a reason for hiding this comment

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

Nav 컴포넌트 파일 잊지말고 리펙토링 한번 다음번에 해주시면 될듯합니다!

수고하셨어요!

(아마 제가 리쿠르팅 페이지 프리페치 박아둬서 충돌날 수 있는데 그 부분만 나중에 머지할때 충돌 해결하심될듯!)

@@ -13,7 +13,8 @@
"next": "14.1.3",
"react": "^18",
"react-dom": "^18.2.0",
"react-icons": "^5.0.1"
"react-icons": "^5.0.1",
Copy link
Member

Choose a reason for hiding this comment

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

https://heroicons.com/

다음번엔 Tailwind 아이콘으로 써드파트 추가해보는건 어떨지 ㅎㅅㅎ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

정말좋습니다

export default function Navigation() {
const pathname = usePathname(); // 현재 경로를 가져오기

const [isProjectOpen, setIsProjectOpen] = useState(false);
Copy link
Member

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.

ok!

@seochan99
Copy link
Member

seochan99 commented May 4, 2024

Screenshot 2024-05-04 at 3 21 30 PM HEY!!!!!!!!!! CHANGE!!!!! PR NAME!!!

@seochan99
Copy link
Member

제가 충돌 수정했음

@shimseohyun
Copy link
Contributor Author

Screenshot 2024-05-04 at 3 21 30 PM HEY!!!!!!!!!! CHANGE!!!!! PR NAME!!!

sorry 경기버스 network error

@shimseohyun shimseohyun merged commit 469f967 into GDSC-DGU:dev May 4, 2024
1 check failed
@shimseohyun shimseohyun changed the title Feature/28 ✨ Feature - Header May 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Header
6 participants