-
Notifications
You must be signed in to change notification settings - Fork 2
리팩토링 규칙
palsa131 edited this page Jan 10, 2022
·
1 revision
components/refactor/atoms/button
- 기존 컴포넌트와 별개 파일로 만들기 → 기존 기능에 영향을 주지 않기 위함 → 제작 후 한 부분에서만 새로운 컴포넌트로 바꿔서 테스트하기 → 완료되면 다른 페이지들은 담당자들이 컴포넌트 확인해서 바꾸기
- 스토리북 만들기(사용방법 문서화) https://storybook-design-system.netlify.app/?path=/docs/button--basic
- 많이 사용되는 컴포넌트부터 리팩토링 시작하기
- 한 컴포넌트에 코드량이 많지 않은가?
- 잘 읽히는가? (다른 사람들이 사용하기에 무리가 없는가?)
- 쓸데없는, 불필요한 로직이 있는가?
- js 내장 함수로 대체할 수 있는 로직이 있는가?
- 중복코드를 줄일 수 있는 방법이 있는가?
고민의 흔적
만난 이슈들
- [소정]emotion theme 적용이 왜 안될까?
- [동진]addEventListener의 passive 속성과 preventDefault
- [동진]storybook - defaultValue is deprecated!
- [소정, 정호]배포시 mapbox가 동작하지 않던 문제!
- [동진]axios interceptor를 설정하면서 만났던 config타입정의 문제
- [정호]Input 값에 undefined나 null이 들어오는 경우 발생하는 오류
- [정호]Javascript에서 KST , TOMORROW 구하는 방법
- [동진]시멘틱한-dropdown만들기!