frontStudySession / jade

0 stars 0 forks source link

form review #1

Open kellywoo opened 7 months ago

kellywoo commented 7 months ago

https://github.com/frontStudySession/jade/blob/main/src/components/Form.tsx#L155C13-L155C67 // 웹에서 email regexp 찾으시면 몇가지가 나올거에요... 위의 regexp 는 '@@@' 도 통과됩니다.

https://github.com/frontStudySession/jade/blob/main/src/components/Form.tsx#L59 // error 객체가 매번 달라지기때문에 해당 부분은 attr의 className 이나 혹은 해당 클래스에 &.is-error 같은 에러 클래스를 추가하고 인풋에 매뉴얼하게 추가하시는 것이 좋습니다.

input.attr(({$error})=> {
  return {className: $error ? 'is-error' : undefined} 
})`
&.is-error {
   ....
}
`

https://github.com/frontStudySession/jade/blob/main/src/components/Form.tsx#L161 // mobile의 경우 숫자만 입력하기 때문에 inputmode="numeric" 넣어주시면 모바일에서 숫자 키패드가 나옵니다. (웹은 적용안됨) // 둘째로 숫자가 아닌 다른 글자들이 입력됩니다. 숫자만 입력가능하도록 작업해보세요.

https://github.com/frontStudySession/jade/blob/main/src/components/Form.tsx#L179 // 같은 register 객체를 써야한다면 하나만 객체를 만들어서 그대로 넣으시는게 더 좋을것 같습니다.

const radioRegister = register('radio', ....)

 <input type=''radio"  {...radioRegister}/>
 <input type=''radio"  {...radioRegister}/>
ansmeer008 commented 7 months ago

우와~~ 리뷰 감사합니다! 말씀해주신 부분 잘 확인해서 적용시켜보겠습니다!❤️‍🔥