21-10-23 회의록

• 공통의 변경사항은 develop 브랜치에 바로 push 하기

• 스타일은 css 랑 js로 나눠서 관리하니까 더 더러워지는 것 같음
• ThemeProvider.js로 하기
• 각자의 규ㅜ격은 각자가 넣기
• fontSize: 정희
• interval(?) margin()? : 경희
• color: 소진

• 42님 코드를 보면 해결은 가능할 것 같다.. 근데.. 지금 여기에 시간을 쏟고 싶지 않다!
• 나중에 하자.

• Prop에서 선협님 코드 기준 horizontal이나 vertical이다
• PropTypes.oneOf(["horizontal", "vertical"])
• PropTypes.oneOfType([PropTypes.number, PropTypes.string])
• https://reactjs-kr.firebaseapp.com/docs/typechecking-with-proptypes.html
• 코드리뷰 때 프롭타입 쪽 봐주기

• react-router-dom 의 <Link> / <a>
• 전자: to="/"
• <Link to={path}><Text>{children}</Text></Link>
• 후자: onClick e.preventDefault / useHisotry().push
• 전자를 선택하는게 좀더 간결할듯!

• Button의 크기가 규격화하기 애매하다
size / fullWidth
• fullWidth :100%(padding-top, 14px)
• padding: 14 24

• StyledIcons
• 색깔 바꾸는 법 참고!
• HTML 구조 확인해서 스타일 먹이기
• https://styled-icons.dev/?s=
• https://github.com/styled-icons/styled-icons

• 아.. styled compoonent에 기본적인 html attrs 반영된다ㅡㄴ 사실을 제가 자꾸잊습니다...여러분 지적해주세요..!
• name, value→ isRequired 여야 하는데! 명시해주세요!!! isRequired로
• <Input {...props}/>

실수해도 괜찮아요

• Pull Request에서 merge가 conflict나서 안된다고 하는경우!
• 일단 develop 브랜치를 pull
• 그러면 conflict가 납니다! 이를 해결해서 commit하고 push 하면 이제 merge 가능!