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

[Refactor] 링크 아이템 컴포넌트 개선 #252

Open
dev-hamster opened this issue Aug 14, 2023 · 0 comments
Open

[Refactor] 링크 아이템 컴포넌트 개선 #252

dev-hamster opened this issue Aug 14, 2023 · 0 comments
Assignees
Labels
♻️ 리팩토링 동작 결과는 같으나 코드 자체 전면 수정 및 성능 개선

Comments

@dev-hamster
Copy link
Member

📌 Feature Issues

현재 링크 컴포넌트의 문제점

  • 강결합된 컴포넌트
    베이스가 되는 LinkItem 에 새로운 기능(버튼)이 추가되는 만큼 props 가 추가 되는 구조입니다. 그래서 props를 많이 내려주는 것이 아닌 합성 컴포넌트로 개선한다면 말랑말랑한 컴포넌트를 만들 수 있을 것 같습니다.
// LinkItemWithProfile
  return (
    <LinkItem
      Header={
        <Profile>
          // ...
        </Profile>
      }
      {...props}
    />
// LinkItem
// 중앙, 하단에 추가해야할 컴포넌트가 생기면 그 수만큼 props가 추가되어야 함
const LinkItem = ({ Header, queryKey, ...props }: LinkItemProps) => {
  return(
    <Wrapper>
      <article>
        {Header}
        <LinkItemInfoWrapper onClick={handleLinkClick}>
         // ...
        </LinkItemInfoWrapper>

        <UtilsWrapper>
          // ...
        </UtilsWrapper>
      </article>
    </Wrapper>
);
}

✨ To-do

  • [ ]
@dev-hamster dev-hamster added the ♻️ 리팩토링 동작 결과는 같으나 코드 자체 전면 수정 및 성능 개선 label Aug 14, 2023
@dev-hamster dev-hamster self-assigned this Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ 리팩토링 동작 결과는 같으나 코드 자체 전면 수정 및 성능 개선
Projects
None yet
Development

No branches or pull requests

1 participant