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

fix: Use styled props in Collapse component #1

Merged
merged 2 commits into from
May 10, 2022

Commits on May 8, 2022

  1. fix: Use styled props in Collapse component

    리액트에서 권장하는 "진리의 원천" 개념을 바탕으로 최소한의 props 의 조합으로 새 속성 값을 할당하는 방법을 사용했습니다:
    
    > _어떤 값이 props 또는 state로부터 계산될 수 있다면, 아마도 그 값을 state에 두어서는 안 됩니다._
    >
    > [교훈, State 끌어올리기 – React](https://ko.reactjs.org/docs/lifting-state-up.html#lessons-learned)
    mu-hun committed May 8, 2022
    Configuration menu
    Copy the full SHA
    21be11c View commit details
    Browse the repository at this point in the history

Commits on May 9, 2022

  1. chore: Rename to CollapseContainer

    @x86chi x86chi yesterday
    
    혹시 RenderComponent 라는 이름이 DOM 커스텀 태그를 위한 네이밍인가요?
    
    제가 "as" polymorphic prop 로 교체하면서 해당 용도가 없어졌는데요, 다른 컴포넌트의 컨테이너 네이밍을 따라 CollapseContainer 로 바꾸면 어떨까 싶습니다..!
    
    @shiftpsh shiftpsh 13 hours ago
    
    괜찮은 것 같습니다. 혹시 그렇게 수정해주실 수 있으신가요?
    
    ref: solved-ac#1 (comment)
    mu-hun committed May 9, 2022
    Configuration menu
    Copy the full SHA
    f946f76 View commit details
    Browse the repository at this point in the history