titicacadev / triple-frontend

Monorepo for Triple Frontend Components and Libraries
https://storybook.triple-corp.com
MIT License
148 stars 9 forks source link

NumericSpinner 컴포넌트를 리펙토링합니다. #1228

Closed rhiokim closed 1 year ago

rhiokim commented 3 years ago

설명

NumericSpinner 는 투어티켓, 지니, 항공 등 몇몇 프로젝트에서 사용중인데요. 이번 투어티켓에 비지니스 로직이 NumericSpinner 에 영향을 주는 요소가 발생하여(https://github.com/titicacadev/tna-web-v2/issues/441) NumericSpinner 에 필수 역할만 유지하고 그외에 요소들(여백설정, 추가정보등)을 TF 에 의존하지 않는 형태로 리펙토링합니다.

NumericSpinner

type Props = {
  /** 최소값 */
  min: number
  /** 최대값 */
  max: number
  /** 초기값 */
  defaultValue?: number
  /** 증가/감소 폭 */
  step?: number
  /** 값 변경 이벤트 */
  onChange: (value: number) => void
}

export default function NumericSpinner({
  min,
  max,
  defaultValue = 0,
  step = 1,
  onChange,
}: Props) {
 return <>..</>
}

<NumericSpinner min={0} max={10} step={2} onChange={...} />

영향을 주는 프로젝트

https://github.com/search?p=2&q=org%3Atiticacadev+NumericSpinner&type=Code

위 3개 정도되는거 같아서 하위호환을 깨트리고 진행해볼까 합니다.

giwan-dev commented 3 years ago

https://github.com/titicacadev/triple-hotels-web/blob/a43456600f3cd5b05ad3502d4df20ead9f511467/src/children-age-input/index.tsx#L4

호텔에서도 사용중입니당!

rhiokim commented 3 years ago

titicacadev/triple-hotels-web@a434566/src/children-age-input/index.tsx#L4

호텔에서도 사용중입니당!

앗 왜 호텔을 뺐을까요 ㅠ