search

kufu / smarthr-ui

React components for creating SmartHR applications.
https://story.smarthr-ui.dev
MIT License
880 stars 141 forks source link

feat(DefinitionList): Composition できるように修正 #5094

Closed uknmr closed 2 weeks ago

uknmr commented 2 weeks ago

関連URL

概要

DefinitionList > DefinitionListItem と Composite できるように修正しました。 ついでに #4949 の方針に沿って、Story を見直しました。

変更内容

確認方法

Storybook や Chromatic で確認してください。

pkg-pr-new[bot] commented 2 weeks ago

Open in Stackblitz

pnpm add https://pkg.pr.new/kufu/smarthr-ui@5094

commit: 0d54cb2

AtsushiM commented 2 weeks ago

@uknmr compositionパターンにするなら dt,dd,dd,... のようにdt一つに対して複数のdd紐づけられる、もしくはdt複数に対して一つのddみたいなパターンも対応できるようにする必要はなさそうですか？ (あんまりなさそうとはいえ、htmlの仕様上はできるわけだし...) https://developer.mozilla.org/ja/docs/Web/HTML/Element/dl

uknmr commented 2 weeks ago

@AtsushiM 装飾として dt と dd が一対一になってしまっているので、需要次第だと思いますが需要もここ5年はないので需要が高まってきたら検討で良さそう。