fix!: TimePicker/MonthPickerにtypeを指定できないようにする - Githubissues

kufu / smarthr-ui

React components for creating SmartHR applications.

https://story.smarthr-ui.dev

MIT License

879 stars 141 forks source link

fix!: TimePicker/MonthPickerにtypeを指定できないようにする #5051

Closed zoshigayan closed 1 month ago

zoshigayan commented 1 month ago

関連URL

無

概要

従来の型定義だと TimePicker / MonthPicker は input 要素に付与可能なすべての属性をpropsとして受け取ります
その結果 <TimePicker type="date" /> といった意図のわからない指定が可能になっていました
- ランタイムでは内部的に指定されている type=time が勝つので当該指定をしても挙動は変わりません
- あくまで「型定義上は可能」という話
TimePicker は既にコンポーネントとして提供されているため一応BREAKING CHANGEという扱いにしています

変更内容

PickerProps から type を除外しました

確認方法

✅ TimePicker / MonthPicker に type 属性を指定すると型検査でエラーとなること

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

Open in Stackblitz

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

commit: 143b5db

zoshigayan commented 1 month ago

ありがとうございます！！