Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 similar comment
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
prgmr99
left a comment
There was a problem hiding this comment.
고생하셨습니다~~!! LGTM👍
코드가 깔끔해서 이해하기 쉬웠슴다 :)
| font-size: ${isDesktop ? "1.6rem" : "1.4rem"}; | ||
| font-weight: ${isDesktop ? "600" : "500"}; | ||
| line-height: ${isDesktop && "140%"}; |
There was a problem hiding this comment.
contents에 Typography를 적용하기에는 쉽지 않을까요?
There was a problem hiding this comment.
엇 제가 의도를 정확히 이해하지 못한 것 같습니다..
There was a problem hiding this comment.
하단에 contents 부분에 반영이 되는 스타일링 같아서요..!
아래와 같이 적용해도 되지 않을까해서 남겨봤습니다!
<Typography variant={isDesktop ? ~~ : ~~} >{contents}</Typography>
There was a problem hiding this comment.
승준님이 추천해주신 부분 말고도
${isDekstop ? css``" : "css``} 형태로도 모바일과 데스크탑 스타일의 구분이 하나의 분기로 명확하게 나눠질 수 있을 것 같아요!
| introduction: string; | ||
| }; | ||
|
|
||
| export const AdvanceQuestionsNum = 2; |
There was a problem hiding this comment.
진행사항, 목표 진행률에 합계를 나타내는 변수입니다!!
| type?: "confirm" | "alert"; | ||
| buttonText?: string[]; | ||
| autoClose?: boolean; | ||
| enableHeader?: boolean; |
There was a problem hiding this comment.
이게 그 데스크탑 모달의 헤더를 지우는 옵셔널로 추가된 부분 맞죠?!
🏄🏼♂️ Summary (요약)
🫨 Describe your Change (변경사항)
/retrospect/writespaceId,retrospectId,title,introduction🧐 전달사항
회고 작성 완료 화면에서는 패딩이 다르게 적용되어 있습니다. 이 부분을 콘텐츠 내부에서 처리하는 것이 좋을지 함께 논의하고 싶습니다.
📚 Reference (참조)
2025-10-17.8.56.11.mov