feat(editor): 에디터 컴포넌트 구현 #17
Conversation
Collection50
requested review from
qkrdmstlr3,
woo-jk and
eunbeann
as code owners
src/components/Editor/Editor.tsx
Outdated
There was a problem hiding this comment.
이 파일이 최종적으로 노출되는 Editor 컴포넌트입니다.
src/app/editor/page.tsx에서 불러오는 것처럼 dynamic lazy loading을 사용하면 번들 사이즈를 줄일 수 있습니다.
There was a problem hiding this comment.
/editor 페이지의 번들 크기를 첨부합니다.
There was a problem hiding this comment.
figma에서 사용되는 아이콘과, lucide-react에서 다른 아이콘을 사용하고 있어서, 에디터 전용 Icon을 만들었는데 어떠신가요??
There was a problem hiding this comment.
shadcn-ui에 기반한 에디터 툴바입니다.
src/components/ui/dropdown-menu.tsx
Outdated
There was a problem hiding this comment.
shadcn-ui에서 설치한 dropdown입니다.
There was a problem hiding this comment.
제가 이전 PR에서 shadcn의 드랍다운메뉴 컴포넌트를 설치했었는데 한번 pull 받고 확인해주세요!
src/components/ui/popover.tsx
Outdated
There was a problem hiding this comment.
shadcn-ui에서 설치한 popover입니다.
현재 리뷰로 충분하다고 생각합니다 👍 |
src/app/editor/page.tsx
Outdated
There was a problem hiding this comment.
(sidebar) 폴더 내부로 이동 시켜야 할 것 같습니다!
| <button ref={ref} disabled={disabled} className={buttonClassName} {...rest}> | ||
| {children} | ||
| </button> |
There was a problem hiding this comment.
여기는 공통 컴포넌트의 Button을 감싸서 사용해도 좋을 것 같아요
그리고 말씀해주신 버튼의 도메인 종속 관련한 문제에 대해서는, 저는 지금 상태로 유지하는게 좋을 것 같습니다!
앞으로 계속 도메인 종속적인 버튼 컴포넌트들이 많이 생길 수도 있는데, 그 때마다 공통 컴포넌트로 분리하기위한 고민을 해야하면 좀 힘들 것 같아요.. 이런 부분은 그냥 종속적으로 만들어도 괜찮을 것 같습니다!
src/components/ui/dropdown-menu.tsx
Outdated
There was a problem hiding this comment.
제가 이전 PR에서 shadcn의 드랍다운메뉴 컴포넌트를 설치했었는데 한번 pull 받고 확인해주세요!
1. 무슨 이유로 코드를 변경했나요?
2. 어떤 위험이나 장애를 발견했나요?
3. 관련 스크린샷을 첨부해주세요.
2024-08-02.10.38.47.mov
4. 완료 사항
5. 추가 사항 / 코드 리뷰 받고 싶은 부분
EditorButton,DropdownButton과 같이 공통 컴포넌트와 유사하나, 에디터에만 종속되어있는 버튼이 존재합니다.globals.css의 위치를styles/globals.css내부로 수정했습니다. 의견 부탁드립니다 !!utils/cn.ts와utils/tailwind-util.ts가 동일한 코드로 보여지는데 어떤 네이밍이 좋으신가요??