커스텀 템플릿 작업 (#450) #562
커스텀 템플릿 작업 (#450) #562
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 |
prgmr99
left a comment
prgmr99
left a comment
There was a problem hiding this comment.
고생하셨습니다..!!
몇가지 검토할 사항들이 있습니다! 확인부탁드려요~
| const setRetrospectValue = useSetAtom(retrospectInitialState); | ||
|
|
||
| const handleSelectTemplate = () => { | ||
| console.log("templateId", templateId); |
| </DropdownMenu.Content> | ||
| </DropdownMenu> | ||
| )} | ||
| <Typography variant={"subtitle14SemiBold"}>{title}</Typography> |
| {!readOnly && ( | ||
| <Button | ||
| colorSchema={"outline"} | ||
| onClick={(e) => { | ||
| e.stopPropagation(); | ||
| navigate(PATHS.retrospectCreate(), { | ||
| state: { spaceId, templateId: id }, | ||
| }); | ||
| }} | ||
| > | ||
| 선택하기 | ||
| </Button> | ||
| )} |
There was a problem hiding this comment.
모바일에서 같이 사용되는 컴포넌트인거 같은데,
이 변경사항은 모바일 환경에서는 문제가 없을까요?
| `} | ||
| > | ||
| <Typography variant="S2">{title}</Typography> | ||
| {readOnly && isLeader && ( |
There was a problem hiding this comment.
이 변경사항은 모바일 환경에서는 문제가 없을까요?
모바일 환경에서도 사용되는 컴포넌트인거 같은데, readOnly && isLeader도 같이 제거되어도 되는지
확인이 필요할거 같아요..!
klmhyeonwoo
left a comment
klmhyeonwoo
left a comment
There was a problem hiding this comment.
이얏 고생하셨습니다! 승준님 코멘트만 확인해주시면 좋을 것 같아요!
|
공통사용되는 컴포넌트임을 확인하고, desktop view, moblie view 일경우 다른 ui 반영해서 커밋해두었습니다! 확인해주셔서 감사합니다! |
|
모바일, 데스크탑 동작 확인했습니다! |
🏄🏼♂️ Summary (요약)
커스텀템플릿 관련 작업입니다
KakaoTalk_Video_2025-10-11-16-19-19.mp4