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: Toggle 컴포넌트 재구축 및 서비스 코드 반영, 디자인시스템 v1.3.10 배포 #793

Merged
merged 6 commits into from
Feb 15, 2024

Conversation

dladncks1217
Copy link
Member

@dladncks1217 dladncks1217 commented Feb 15, 2024

📄 Summary

기존 토글의 경우 사용하는 측에서 useSelect를 받아와 사용해야 했음.
사용하는 측에서 컴포넌트가 너무 무거워지고 복잡해짐. 개선을 위한 재구축

기존 사용의 경우 아래와 같았음.

const { selected, handleSelectClick } = useSelect(ORDER_BY_REGISTRATION);
    // -> props로 내려줌

const 컴포넌트명 = ({selected, handleSelect}) =>{
return(
     <ToggleGroup>
         <Toggle selectedId={selected} changeSelect={handleSelect}>
         // ...
      </ToggleGroup>
}

개선 이후

   <NewToggle additinalFunc={함수명} {...args}>
      <h6>NewToggle</h6>
      <div style={{ display: 'flex' }}>
        <NewToggle.List text="Toggle 1" toggleKey="toggle1" />
        <NewToggle.List text="Toggle 2" toggleKey="toggle2" />
      </div>
      <div>
        <NewToggle.Item toggleKey="toggle1">나는토글1</NewToggle.Item>
        <NewToggle.Item toggleKey="toggle2">나는토글2</NewToggle.Item>
      </div>
    </NewToggle>

이름은 기존 쓰고있던것들때문에 임시로 NewToggle로 해뒀습니다
NewToggle안에 additinalFunc는 내부적으로 변화하는 state에 의해 외부에서 컨트롤해야할 뭔가가 필요할때 쓰면 됩니다.

additinalFunc={(id: number | string) => {console.log(id)}} 
// 예시이긴 한데, id로 toggleKey가 들어온다고 생각하고 쓰면 됨

🙋🏻 More

일단 서비스측에서 사용하던 코드 가볍게 조금 고쳤는데, 추가적으로 고쳐갈 예정.

#792

@dladncks1217 dladncks1217 added the FE-DesignSystem 행록 디자인시스템 label Feb 15, 2024
@dladncks1217 dladncks1217 self-assigned this Feb 15, 2024
@dladncks1217 dladncks1217 added the FE-Service 행록 서비스 프론트엔드 label Feb 15, 2024
@dladncks1217 dladncks1217 changed the title refactor: Toggle 컴포넌트 재구축 및 npm v1.3.9 배포 refactor: Toggle 컴포넌트 재구축 및 서비스 코드 일부 반영 Feb 15, 2024
Copy link

로컬 cypress 테스트가 실패했습니다. 테스트 결과 자세히 보기

Copy link

로컬 cypress 테스트가 실패했습니다. 테스트 결과 자세히 보기

Copy link
Collaborator

@LJW25 LJW25 left a comment

Choose a reason for hiding this comment

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

훌륭합니다 리더 슬링키

@dladncks1217 dladncks1217 changed the title refactor: Toggle 컴포넌트 재구축 및 서비스 코드 일부 반영 refactor: Toggle 컴포넌트 재구축 및 서비스 코드 반영, 디자인시스템 v1.3.10 배포 Feb 15, 2024
@dladncks1217 dladncks1217 merged commit cdaa039 into develop Feb 15, 2024
2 checks passed
hgo641 pushed a commit that referenced this pull request Apr 23, 2024
* refactor: Toggle 컴포넌트 재구축 및 npm v1.3.9 배포

* refactor: toggle 변경 시 발생시킬 이벤트 코드 추가

* fix: cyprses CI 버전문제 개선을 위한 package.json 수정

* chore: Toggle style Radius 추가

* build: 디자인시스템 npm v1.3.10 배포

* chore: 불필요한 콘솔 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE-DesignSystem 행록 디자인시스템 FE-Service 행록 서비스 프론트엔드
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants