전남대 FE_김현채_1주차 과제 #32
Conversation
|
| /** | ||
| * Is this the principal call to action on the page? | ||
| */ | ||
| primary: PropTypes.bool, |
There was a problem hiding this comment.
primary 값은 불리언이 아니라 enumable 한 형태로 받는게 좋아보입니다.
eg) variant: "priamary" | "secondary"
|
|
||
| interface InputProps extends React.InputHTMLAttributes<HTMLInputElement> { | ||
| customSize?: 'small' | 'medium' | 'large' | 'responsive'; | ||
| disabled?: boolean; |
There was a problem hiding this comment.
disabled 는 기본적으로 가지는 어트리뷰트인데 따로 추가해준 이유가 궁금합니다.
안녕하세요. 멘토님! 전남대학교 김현채 입니다. 피드백을 위해 시간 내주셔서 감사합니다!
Q1. eslint를 추가하고 Lint 룰을 세팅한다는 개념에서 가장 어려움을 느꼈던 것 같습니다. 이 부분이 올바르게 설정된 건지 궁금합니다! 또한 멘토님의 Lint 룰 세팅 방법을 공유해주신다면 감사드립니다.
Q2. 폴더의 갯수를 늘려 기능 별로 파일들을 관리하는게 좀 더 이상적인 방법일까요? 검색을 통해 이상적인 폴더 관리에 대해 찾아보았는데 아직은 감이 잡히지 않는 것 같습니다..!
Q3. 커밋을 파일 한 개를 수정할 때마다(작은 단위로 꼼꼼히) 하는 것이 좋을지, 좀 더 유연하게 진행해도 좋을지 궁금합니다!(아직은 커밋을 하는게 쉽지 않아 놓치는 경우가 발생하여 여쭤보고 싶습니다.)
이외에도 잘못된 부분이나 수정이 필요한 부분을 알려주신다면 부족하지만 최대한 열심히 받아들이고 고쳐나가겠습니다!!