[🎨 Design System] Storybook showcase + theme switching#1904
[🎨 Design System] Storybook showcase + theme switching#1904
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Storybook Deployed🔗 Preview: https://chainapsis.github.io/keplr-wallet/storybook/ Last updated: 2026-03-30T07:45:25Z |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e5e2b4e731
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| export const Overview: StoryObj = { | ||
| name: "Overview", | ||
| render: () => ( | ||
| <div |
There was a problem hiding this comment.
Import React in new Storybook JSX files
These new Storybook files render JSX without importing React (components.stories.tsx, color.stories.tsx, icon.stories.tsx, and .storybook/preview.tsx), but this package still uses the classic runtime (packages/design-system/tsconfig.json sets "jsx": "react"). In that mode, yarn workspace @keplr-wallet/design-system typecheck fails with TS2874 (React must be in scope), so this commit introduces a typecheck break for the design-system workspace unless each JSX file imports React (or the tsconfig runtime is migrated).
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
stories 3개 + preview.tsx에 import React 추가했습니다. typecheck 통과 확인.
- Color / Typography / Icon 쇼케이스 stories - addon-themes: 툴바에서 Light/Dark 전환 + 배경색 자동 전환 - preview: DSThemeProvider + DSColor.background.surface.ground 배경 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Configure Vite esbuild to use classic JSX transform to match tsconfig's jsx: "react" setting. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…stency Also restore DSTypographyTokensKey reference to match current tokens file. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
bbb7395 to
3a8dc7b
Compare
…n stories Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ff76c838a7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "react@npm:^16.8.0 || ^17.0.0 || ^18.0.0 || ^19.0.0": | ||
| version: 19.2.4 | ||
| resolution: "react@npm:19.2.4" |
There was a problem hiding this comment.
Align Storybook docs with workspace React major
This lockfile change introduces a second React major (19.2.4) for the Storybook docs dependency range while the repo is still pinned to React 18, so Storybook can end up loading docs blocks with a different React instance than the preview renderer. In that mixed-major setup, docs pages commonly fail at runtime with Invalid hook call when rendering stories/components together, which would break Storybook preview/deployment for this package.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
수정했습니다. root resolutions에 react/react-dom을 ^18.2.0으로 추가하여 18.3.1로 통일했습니다.
→ 1aa6445
Add react/react-dom resolutions to pin workspace to React 18, preventing yarn from resolving addon-docs peer dep to React 19. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
| storySort: { | ||
| order: [ | ||
| "Foundations", | ||
| ["Color", "Icon", "Typography", ["Docs", "Showcase", "Playground"]], |
There was a problem hiding this comment.
실제 export 이름은 Typography로 되어있는데 여기는 Showcase로 되어 있어서 매칭이 안되네요. 사이드바에서 무시가 되기는 하는 거 같은데 그래도 매칭시켜주는 게 좋을 거 같습니다.
There was a problem hiding this comment.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
#1900 과 동일한 내용입니다. base 브랜치가 먼저 머지되면서 누락되어 다시 올립니다.
추가 수정
import React누락 수정jsx: "react"와 일치)Showcase→Typography로 변경 (Color, Icon과 URL 패턴 통일)🤖 Generated with Claude Code