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

Feat: home 페이지 구현 #63

Merged
merged 8 commits into from
Mar 14, 2024

Conversation

hankim0904
Copy link
Collaborator

@hankim0904 hankim0904 commented Mar 14, 2024

📌요구사항

  • home 페이지 구현
  • api 명세에 맞게 Card 컴포넌트 및 관련 컴포넌트 수정
  • stacks 컴포넌트에서 Stack 컴포넌트 분리

📌작업 진행 상황 (에러, 막혔던 부분, 그외 궁금한것 등등)

home 페이지 구현

  • 반응형으로 구현
  • /src/constants 에 banners.ts (배너 이미지 및 클릭 시 이동 경로 정보 관리),
    hotAndNewList.ts (home의 4가지 카테고리 관리) 생성
  • isSidebarOpen 상태 관리 코드 필요(zustand)
  • 더보기 눌렀을 때 솔트 적용한 채로 페이지 이동하게 하는 로직 필요
  • 데스크탑 화면에서 사이드바가 열렸을때 어색한 부분 있음 -> 논의 필요
  • 사이드바, 칩 간의 z-index 이슈 있음-> 코드 수정 필요

Card 컴포넌트 수정

  • 업데이트 된 api 명세 반영(Card, Stacks, Positions 컴포넌트 수정)
  • isSidebarOpen prop 삭제
  • /src/lib/mock 경로에 mock 데이터 생성
  • 스크랩 여부에 대한 정보는 전역으로 상태관리 해야 할 것 같음 -> 논의 필요

Stack 컴포넌트 구현

  • /src/constants 에 stackIcons.ts (기술스택 이미지 및 정보 관리) 생성
  • 기술 스택 이름을 prop으로 내려주면 됨

📌스크린샷 / 테스트결과 (선택)

  • home 페이지 반응형
home_.mp4
  • isSidebarOpen true 시 화면 어색함

image

  • chip 및 sidebar z-index 이슈

image

  • Stack 컴포넌트

image

📌이슈 번호

close #42

Copy link

올바른 PR 형식

올바른 PR 형식입니다.
코드 작성하시느라 수고 많으셨습니다 :)

@hankim0904 hankim0904 self-assigned this Mar 14, 2024
@hankim0904 hankim0904 added the enhancement New feature or request label Mar 14, 2024
}

/**
*
* @property width - px단위
* */
export default function Scrap({ isScraped = false, width }: ScrapProps) {
export default function Scrap({ wasScraped = false, width }: ScrapProps) {
Copy link
Contributor

Choose a reason for hiding this comment

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

isScraped 가 wasScraped 로 변경된 이유가 useToggle 에 isScraped 때문인가요??

Copy link
Contributor

@kimbobby kimbobby left a comment

Choose a reason for hiding this comment

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

😊

@hankim0904 hankim0904 merged commit fd91056 into develop Mar 14, 2024
1 check passed
@kimbobby kimbobby deleted the feature/issue-42--Feat-Home-페이지-구현 branch March 14, 2024 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] : Home 페이지 구현
4 participants