search

kufu / smarthr-ui

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

fix!: DropdownScrollArea と DropdownContent の scrollable を削除 #5000

Closed uknmr closed 1 month ago

uknmr commented 1 month ago

関連URL

https://smarthr.atlassian.net/browse/SHRUI-678

概要

DropdownScrollArea を挟まないとスクロールできない、という仕様や DropdownContent の scrollable オプションは「スクロールできないと内包要素に到達できなくなる」可能性があるため削除します。

DropdownScrollArea の存在をしらず、ドロップダウンパネルを超えて表示されてしまう不具合も発生しやすいです。 そもそもどんな場合でもはみ出てしまうのであれあばスクロールを発生させれば良いです。

DropdownScrollArea は利用箇所を純粋に消せば問題ありません。scrollable props はプロダクトからの利用も片手で数えるほどしかなく、特に対応も必要ありません。

変更内容

確認方法

Storybook や Chromatic で問題ないことを確認してください。

pkg-pr-new[bot] commented 1 month ago

Open in Stackblitz

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

commit: c990df9