Skip to content

feat(List): Listコンポーネントを追加#6391

Draft
Qs-F wants to merge 6 commits into
masterfrom
feat/list
Draft

feat(List): Listコンポーネントを追加#6391
Qs-F wants to merge 6 commits into
masterfrom
feat/list

Conversation

@Qs-F

@Qs-F Qs-F commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

関連URL

概要

変更内容

  • マークアップは迷ったんですが、 ul>li>Heading にしました。テーブルのTh[scope=row]相当の動きをさせようと思うと、Headingしかないかなぁと。
    • その代わり、リストのまとまりは読み上げられるためSectionで書くと冗長になると考えSectionは省きました。

プロダクト側で対応が必要な事項

確認方法

https://63d0ccabb5d2dd29825524ab-cxpeuntsub.chromatic.com/?path=/docs/components-list--docs

@pkg-pr-new

pkg-pr-new Bot commented Jun 15, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@smarthr/smarthr-ui-charts@6391
npm i https://pkg.pr.new/smarthr-ui@6391

commit: 928e713

@AtsushiM

Copy link
Copy Markdown
Member

これDefinitionListとの使い分け難しそう...

スクリーンショット 2026-06-16 6 50 48

@Qs-F

Qs-F commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

@AtsushiM 基本的にオブジェクトの羅列とプロパティの羅列で明確に使い分けできると思うので、デザインシステムのガイドラインで縛りたいですねぇ(smarthr-uiでどうにかするのではなく

@AtsushiM

Copy link
Copy Markdown
Member

縛れるんですかねこれ。
見た目が似すぎてるのと、名前がListなこともあって使い分けの判断めっちゃ難しそうな気がします。
なんかこう、利用する目的に合わせた名前?とかあればよさそうな気がするんですが....

@AtsushiM

Copy link
Copy Markdown
Member

FormControlとFieldsetみたいに見た目はほぼ同じでも名前でちゃんと使い分けする場面はわかる、みたいな...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants