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: add List component #1343

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

jinhyeonkwon
Copy link
Contributor

@jinhyeonkwon jinhyeonkwon commented Jan 15, 2025

요약 *

It closes #1342

기존에 여러 구현 방식으로 만들어져 있던 bullet list와 number list를 하나의 컴포넌트로 통일합니다.
List component의 props 정의를 보시면 쉽게 사용이 가능할 것 같습니다.
bullet point로도 사용할 수 있고, 시작 index (0 이상)를 정해서 number list로도 사용할 수 있습니다.
활동보고서 상세 보기 (packages/web/src/features/activity-report/frames/ActivityReportDetailFrame.tsx) 에 우선 적용해 두었으며, 추후 적용이 필요한 다른 페이지 리팩토링을 진행하고자 합니다.

스크린샷

  • Bullet list
    image
  • Number list
    image

data로 string이랑 일반적인 ReactNode를 섞어 쓸 수 있게 허용하다 보니, number로 했을 때 이런 광경을 볼 수 있긴 합니다
image
List 여러 개로 쪼개고, 필요한 대로 startIndex 조절하면서 해결이 가능하나, 추후에 수정 필요가 있다면 numbering 로직을 바꾸겠습니다.

이후 Task *

  • 다른 페이지에 적용하기
  • Storybook 문서화

@jinhyeonkwon jinhyeonkwon requested a review from babycroc January 15, 2025 11:56
@jinhyeonkwon jinhyeonkwon self-assigned this Jan 15, 2025
@jinhyeonkwon jinhyeonkwon linked an issue Jan 15, 2025 that may be closed by this pull request
2 tasks
Copy link
Contributor

Choose a reason for hiding this comment

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

List 폴더 만들거면 ListItem도 이 폴더에 옮기면 좋을 것 같아요

@wjeongchoi
Copy link
Contributor

number 쓰이는 경우가 꽤 있다면 넘버링 로직 수정해보면 어떨까요 @babycroc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: List Component
2 participants