voyagegroup / ingred-ui

https://ingred-ui.netlify.app
MIT License
53 stars 10 forks source link

Bugfix: Tooltipの型情報を修正 #1629

Closed toshi17 closed 3 months ago

toshi17 commented 3 months ago

概要

HTML要素のattributesにはcontentが元々存在していることで、TooltipのPropsであるconentと競合が発生しており、誤った型情報となっている

&を用いた交差型ではプロパティが競合した場合に型同士の積を取るので、本来的にはReactNode型を許容するはずのcontentが string のみを許容するようになってしまっている

やったこと

Tooltipにおける競合するプロパティをomitした上で、交差を取るように

changeset-bot[bot] commented 3 months ago

🦋 Changeset detected

Latest commit: 9a867d5bcf75da4c46ff1867d91db9db2155f8d9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | --------- | ----- | | ingred-ui | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

netlify[bot] commented 3 months ago

Deploy Preview for ingred-ui ready!

Name Link
Latest commit 9a867d5bcf75da4c46ff1867d91db9db2155f8d9
Latest deploy log https://app.netlify.com/sites/ingred-ui/deploys/665d2a8431382700083c23b8
Deploy Preview https://deploy-preview-1629--ingred-ui.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.