refactor(DialogOverlap): useEffectの依存配列からchildrenを削除

[AtsushiM]

関連URL

なし

概要

eslint-config-smarthr@14.7.0で追加されたsmarthr/best-practice-for-unstable-dependenciesルールへの対応として、DialogOverlapコンポーネントのuseEffectの依存配列からchildrenを削除します。

変更内容

• childrenをrefで保存し、毎レンダリング時に最新の値を更新
• useEffectの依存配列からchildrenを削除し、isOpenのみにする
• 不要な再実行を防止しつつ、動作は変わらず

実装の詳細

// childrenをrefに保存（毎レンダリング時に最新の値を設定）
const childrenRef = useRef(children)
childrenRef.current = children

useEffect(() => {
  if (isOpen) {
    setChildrenBuffer(childrenRef.current)
  }
}, [isOpen])

DialogOverlapは、ダイアログが閉じる際のアニメーション中に表示するためにchildrenをバッファリングするコンポーネントです。isOpenがtrueの時に最新のchildrenをバッファに保存し、閉じる際にそのバッファを使用します。

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

なし

確認方法

• 既存のテストが通ることを確認
• Storybookで動作確認

[pkg-pr-new]

Open in StackBlitz

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

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

commit: 08ea6ac