PCWEB-11978 ConfirmModal의 title props의 타입을 변경합니다. #114
Conversation
| @@ -51,7 +51,7 @@ export interface ConfirmModalProps | |||
| > { | |||
| icon?: IconType; | |||
| type?: Type; | |||
| title?: string; | |||
| title?: string | JSX.Element; | |||
There was a problem hiding this comment.
사용되는 곳이 career-web, profile-web, foa-v2, kairos 인데
모두 둘러봤을 때 career-web에서만 JSX를 넘기고 있기도 하고, deprecated될 부분이라 ReactNode타입까지는 굳이? 라는 생각이 들긴 했는데욥... 혹시 모르니 추가하는게 좋겠네요
There was a problem hiding this comment.
이미 머지가 되었지만... ReactNode에 JSX.Element가 포함되어 있지 않나요? @KimJeongHyun
There was a problem hiding this comment.
@saengmotmi
확인해보니 ReactNode에는 ReactElement가 포함되어있고, JSX.Element는 ReactElement를 확장한 타입이네용
declare namespace React {
type ReactNode =
| ReactElement
| string
| number
| ReactFragment
| ReactPortal
| boolean
| null
| undefined;
}여기서 구현상으로 같은 것은 ReactElement인데, JSX.Element와 다른 것은
JSX.Element는 ReactElement의 type과 props를 모두 any로 받게끔 확장된 global namespace 타입이에요.
https://www.totaltypescript.com/jsx-element-vs-react-reactnode
https://www.howdy-mj.me/react/react-node-and-jsx-element
There was a problem hiding this comment.
네 근데 제 생각엔 JSX.Element(JSX 문법은 리액트의 전유물이 아니니...)를 리액트에서 쓰기 위해 한번 더 덮어 씌운게 ReactElement이고 이 타입이 ReactNode에 포함 되어 있으니, 그리고 ReactNode로 쓴다고 해서 별 다를 것도 없으니 굳이 유니온으로 표현해서 얻을게 없지 않나 하는 의견이었어요 @KimJeongHyun
There was a problem hiding this comment.
@saengmotmi 배포해서 확인해보니, string도 타입에 넣을 필요가 없었군요, ReactNode 안에 있으니
겸사겸사 그냥 ReactNode 받는걸로 하겠슴다.
작업 내용 (Content) 📝
ConfirmModal의 title 타입을 string에서 JSX.Element도 받을 수 있도록 변경합니다.
career-web의 ts migration시, string만이 아닌 JSX.Element도 넘어가도록 작성된 js 파일들이 있어서 해당 케이스에 대한 대응 작업입니다.
스크린샷 (Screenshot) 📷
링크 (Links) 🔗
기타 사항 (Etc) 🔖
테스트 Checklist (Test Checklist) ✅
희망 리뷰 완료 일 (Expected due date) ⏰
202X.X.X. Wed