Repository navigation
🐛 fix(card): 狭い幅で CardTitle が縦書きになる不具合を直す(v1 backport) - #373
Conversation
- CardHeader / CardTitle を flex-wrap にし、入りきらないときは CardControl と CardDescription を次の行に折り返す - 折り返した CardControl を ml-auto で右端に寄せる - 見出しの横に Tag と文字のボタンを並べる組み合わせを CardHeader の JSDoc アンチパターンに追加 - 幅 260px で見出しが 1 行に収まることを確かめる NarrowHeader ストーリー(play)を追加 - CHANGELOG に追記 Closes #371 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016SVVD5E6nsRMU6HXqyY3R1 (cherry picked from commit 3e9b91d)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are reported for the Card layout backport addressing narrow-width Japanese heading wrapping.
No open issues 🎉
What changed in this PR
#372 の Card レイアウト修正を、React 18 の 1.x メンテナンスラインへ backport する PR です。既存の React.forwardRef を維持し、狭い幅で日本語の見出しが 1 文字ずつ折り返される問題(#371)に対応します。
Changes:
- CardHeader / CardTitle に折り返しを追加し、CardControl を右寄せに変更。
- レイアウトクラスの単体テストと、幅 260px で見出しが 1 行に収まることを検証する Storybook ストーリーを追加。
- CHANGELOG の Unreleased / Fixed に修正内容を記載。
| File | Description |
|---|---|
| src/components/ui/card/index.tsx | ヘッダーとタイトルの折り返し、操作領域の右寄せを追加。 |
| src/components/ui/card/index.test.tsx | 追加したレイアウトクラスのアサーションを追加。 |
| src/components/ui/card/index.stories.tsx | 狭幅での見出し表示を検証する NarrowHeader を追加。 |
| CHANGELOG.md | 狭幅での CardTitle 表示不具合の修正を記録。 |
Review effort: Balanced
🤖 Generated with Claude Code
https://claude.ai/code/session_016SVVD5E6nsRMU6HXqyY3R1
概要
#372 の修正を 1.x(React 18 メンテナンスライン)に backport します。CardHeader の幅が足りないと、日本語の CardTitle が 1 文字幅まで縮んで縦書きになる不具合です(#371)。
git cherry-pick -x 3e9b91dで取り込みました。v1 の Card はReact.forwardRefのままなので、衝突したindex.tsxはクラス名の変更だけを手で当てています(ref-as-prop は持ち込んでいません)。変更内容
flex-wrapに、CardControl にml-autoを追加NarrowHeaderストーリー(play)を追加コミットメッセージには「JSDoc アンチパターンに追加」とありますが、main 側でガイドラインに合わせて取り下げた(#372 の 7634407)ので、この PR には JSDoc の変更を入れていません。amend + force-push はしない方針のため、メッセージはそのままです。
動作確認
pnpm lint(warning のみ)/ prettiervitest --project unit src/components/ui/card: 33 passedvitest --project storybook src/components/ui/card: NarrowHeader を含む 4 件 passed@vitest/browser-playwright4.1.8 がずれていて、Snapshotsの export が無いという unhandled error が 1 件出ます。この PR とは関係なく、v1 の既存の状態です関連 Issue
Generated by Claude Code