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] Project Feedback 반영, 모션 라이브러리 적용 #37

Merged
merged 12 commits into from
May 4, 2024

Conversation

naarang
Copy link
Contributor

@naarang naarang commented May 1, 2024

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


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

개요

맡은 페이지 : 프로젝트 목록 페이지, 솔루션 챌린지 목록 페이지, 프로젝트 상세 페이지

  • 지난번 피드백 반영 및 모션 라이브러리를 추가로 적용했습니다.

변경 사항

  • 하드코딩 개선해서 더미데이터 constant 폴더로 분리
  • Font, Color, Breakpoint custom style 수정

프로젝트 카드 2줄 이상일 경우 ... 추가
스크린샷 2024-05-01 오후 10 56 56

  • [🥹 Feedback] Project Feedback #26 task 완료(디자인 수정 및 토글 구현)
  • framer-motion 라이브러리 적용하기(우선은 제가 맡은 프로젝트, 솔루션 챌린지 부분에 적용했습니다!)

코드 리뷰시 참고 사항

  • 프로젝트 목록 페이지의 경우 desktop 크기에서 페이지네이션, tablet/mobile 크기에서 무한 스크롤을 사용하는데 이를 어떻게 분리할까 고민하다가 우선은 아예 컴포넌트를 분리해서 hidden을 적용했습니다.. 혹시 더 좋은 방법이 있을까요?
  • 모션 적용을 하면서 더 예쁘게 만들지 못한다는 아쉬움을 많이 느껴서ㅠㅠ 모션 효과 좋은 아이디어 말씀해주시면 적용해보겠습니다!

@naarang naarang self-assigned this May 1, 2024
@naarang naarang added 🎨 Design 마크업 & 스타일링 ✨ Feature 기능 개발 🙋‍♀️ 나경 작업 labels May 1, 2024
@naarang naarang changed the title Feature/project page [✨Feature] Project Feedback 반영, 모션 라이브러리 적용 May 1, 2024
@naarang naarang added the 🔨 Refactor 코드 리팩토링 label May 1, 2024
@naarang naarang requested review from shimseohyun, seochan99, bunju20, dlawjddn, bianbbc87 and puretension and removed request for dlawjddn May 1, 2024 13:55
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.

저도 페이지네이션 나경님이랑 비슷한 방법으로 했어요! css로 구분해야하는거라..ㅠㅠ

저는 데이터를 현재 선택한 페이지에 대한 세미나 12개 데이터랑 전체 데이터로 분리해서

  1. desktop div - 세미나 12개 데이터로 map(tablet, mobile:hidden)
  2. tablet, mobile div - 전체 데이터로 map(desktop:hidden
    위처럼 2개로 나누고 페이지네이션 부분도 tablet, mobile 때에는 hidden으로 나오게 했어요!

수고하셨습니다!!

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.

고생하셨습니다 ㅎㅎ

모션은 한번 생각해봐야겠네요! 고생많았어요!

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.

고생하셨습니다!!

@naarang naarang merged commit 45a40da into dev May 4, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Design 마크업 & 스타일링 ✨ Feature 기능 개발 🔨 Refactor 코드 리팩토링 🙋‍♀️ 나경 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants