Skip to content

Conversation

@suji8073
Copy link
Member

@suji8073 suji8073 commented Jun 16, 2025

Motivation 🤔

  • 상위 리스트 페이지 중 TopList 컴포넌트 구현
  • TopList 스토리북 구성

Key Changes 🔑

const ListPage = () => {
  // 서버에서 받아오는 값
  const currentCount = 0;
  const totalCount = 8;

  return (
    <div className='h-full w-full overflow-hidden bg-gray-50 text-center'>

      {/* inProgress */}
      <TopList label='text' progressText={`${currentCount}/${totalCount}`} />

      {/* // 완료 */}
      <TopList label='text' />

    </div>
  );
};

export default ListPage;
  • Folder.svg

    • PNG 사용 시 화질 저하로 임시 SVG 적용
    • 해당 부분이 이미지로 교체되는 지에 대한 여부는 피그마를 통해 문의
  • TopList 내 props

    • currentCount, totalCount props를 각각 넘기는 방식으로 진행

Attachment 📷

image

@suji8073 suji8073 self-assigned this Jun 16, 2025
@suji8073 suji8073 requested a review from jungjunhyung99 as a code owner June 16, 2025 15:34
@suji8073 suji8073 added the ✨ Feature 기능 개발 label Jun 16, 2025
Comment on lines 4 to 5
label: string;
progressText?: string; // '0/8' or undefined
Copy link
Contributor

Choose a reason for hiding this comment

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

오 저는 요 부분 총 일정 개수와 완료된 일정 개수가 number 타입으로 내려오고, 총 일정 개수와 완료된 일정 개수가 같아지면 완료 를 렌더링 하면 될 줄 알았는데... 어떻게 될 지 모르겠네요...! 추후 의논해보는 거 어떠신가요?!

Copy link
Member Author

Choose a reason for hiding this comment

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

헉 ! 제가 생각했을 때도 그게 맞는 것 같네용 ! 수정하겠습니다 ~~

@suji8073 suji8073 changed the base branch from main to develop June 18, 2025 13:21
suji8073 added 3 commits June 18, 2025 22:22
- folder.svg 추가(임시)
- progressText와 같이 가공된 문자열이 아닌 각각 Number 타입으로 props 전달
@suji8073 suji8073 force-pushed the f/top-list-component/suji_chae branch from 401a53d to 3bcc8b5 Compare June 18, 2025 13:22
@suji8073 suji8073 merged commit 93de74c into develop Jun 18, 2025
1 check passed
@github-actions
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants