Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
|
Alpha Preview (Docs)
|
Alpha Preview (Stackflow SPA)
|
Alpha Preview (Storybook)
|
10641ed to
9513f74
Compare
|
데스크톱 컴포넌트를 같은 층위에 밀어넣는게 좋은 선택같지는 않네요. stackflow처럼 패키지라도 분리하는게 어떨까 싶긴 합니다. |
856568e to
f3458b7
Compare
|
@malangcat action-button과 같이 데스크탑에서도 사용되고, 모바일에서도 사용되는 컴포넌트는 그대로 두되, 아마 애셔가 얘기해주신 부분이 최소한의 맥락 분리라고 예상하긴한데, (stackflow를 언급해주신 부분에서 그런 생각이 들었습니다.)
같은 패키지에 두는 건 왜 좋은 선택이 아닌지에 대한 의견이 더 궁금해요 애셔 |
|
음... 처음 빌드할때 프로젝트의 목표 자체가 웹뷰에 들어갈 모바일 제품만을 위한 것이였어서, react라는 패키지명을 사용했는데 데스크톱 관심사를 포함하게 되면 패키지명이 애매해지긴 하네요. 저라면 rootage 스펙부터 전부 따로 구성할 것 같긴 합니다. 결국 데스크톱 관심사에서는 typography부터 컴포넌트들의 사이즈 요구사항들이 전부 달라지게 될 가능성이 커서요. ecosystem은 어떤 시스템을 만들더라도 공유될 수 있는 영역, packages는 모바일 제품, 즉 당근 앱을 위한 영역인 것이 의도였습니다. |
|
@malangcat 웹 디자인시스템 rootage - recipe - react의 레이어를 따로 구성하는건 동의해요. 저는 문서 페이지 때문에 한 레포에 다 담아야 관리가 쉬울 것 같은데, 근데 현재의 모노레포 구조도 조금 애매하네요.
|
packages/ 안에서 다시 mobile, desktop으로 분류하거나... 패키지명은 저도 모르겠네요 완전히 모바일만을 전제로 했던 네이밍이라 ㅋㅋㅋㅋ |
|
@malangcat ㅋㅋ 그래도 같이 얘기해주셔서 감사합니다. 그래도 어느정도 감이 잡히긴 했어요 |
91c7cbd to
9b3802f
Compare
f7abd57 to
6f2415a
Compare
…d block examples - Add Header slot recipe (root/left/center/right) with size, transparent, divider variants - Add HeaderActionButton and HeaderToggleButton as independent qvism preset recipes - Add React components: Header.Root, Header.Left, Header.Center, Header.Right, Header.ActionButton, Header.ToggleButton - Add 4 block examples: 3 generic company headers + 1 Daangn-branded header - Register header recipes in qvism-preset and generate CSS outputs Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
f50849a to
3636cba
Compare
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- header-05: responsive nav with desktop links + mobile BottomSheet menu - header-06: transparent header with scroll-driven background transition Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…av, section) Primitive.header was undefined, causing Header component to throw TypeError during Next.js static export pre-rendering, which resulted in all header block pages being baked with 404. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…omponent # Conflicts: # packages/css/all.layered.min.css # packages/css/all.min.css
Move Menu dropdown examples (Platform, Features) into header-03 and remove the separate header-05 responsive block. Fix MenuContentProps missing children type in menu registry UI. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Summary
Root/Left/Center/Right3-slot compound component)pull-to-refresh선례)tone(layer | transparent),divider(boolean)Test plan
bun qvism:generate— CSS 생성 확인bun packages:build— 빌드 성공bun test:all— 585 pass, 0 fail🤖 Generated with Claude Code