Doist / reactist

Open source React components made with ❤️ by Doist
http://doist.github.io/reactist
MIT License
247 stars 21 forks source link

fix: Explicitly omit `crossOrigin` and `placeholder` props #812

Closed pawelgrimm closed 7 months ago

pawelgrimm commented 7 months ago

Relates to:

Short description

Due to an update to @types/react, certain intrinsic HTML element type have been updated to remove unsupported props, namely:

This means that if a consumer of this package were to upgrade their @types/react, while this package retains the older types, usages of components such as <TextInput> and <Heading> would be met with type errors complaining about missing those props that have been removed.

The more correct solution here would be to update the @types/react package in THIS package, we cannot do so because it causes similar errors stemming from ariakit, which is not trivial to upgrade.

As an interim solution, we are explicitly excluding those removed props from affected Reactist component props.

PR Checklist

Versioning

This is technically a breaking change, as the interface of many components has changed.