Skip to content

Conversation

@DASPRiD
Copy link
Contributor

@DASPRiD DASPRiD commented Mar 30, 2022

There is a known issue with storybook that the @emotion/core dependency augments the React HTMLAttributes with an additional css property. This causes the compiled types of react-bootstrap-typeahead to include that attribute as well, which in turns breaks Typescript compilation of projects using it.

See: emotion-js/emotion#1257 (comment)

If possible, please release this in an alpha.10 package soon, as it is currently stopping one of our projects from building.

There is a known issue with storybook that the @emotion/core dependency augments
the React `HTMLAttributes` with an additional `css` property. This causes the
compiled types of react-bootstrap-typeahead to include that attribute as well,
which in turns breaks Typescript compilation of projects using it.

See: emotion-js/emotion#1257 (comment)
@ericgio
Copy link
Owner

ericgio commented Mar 30, 2022

Thanks @DASPRiD!

@ericgio ericgio merged commit 302d813 into ericgio:main Mar 30, 2022
@ericgio
Copy link
Owner

ericgio commented Mar 30, 2022

Released as part of v6.0.0-alpha.10

@DASPRiD DASPRiD deleted the fix/remove-emotion-augmentation branch March 30, 2022 10:23
@DASPRiD
Copy link
Contributor Author

DASPRiD commented Mar 30, 2022

Released as part of v6.0.0-alpha.10

I think you might not have run yarn before publishing? I still see the css property in the resulting d.ts files.

@ericgio
Copy link
Owner

ericgio commented Mar 30, 2022

@DASPRiD where are you seeing the declarations show up? Are they actually output in the package files or just implicitly pulled in and applied when typechecking your code? Thanks.

@ericgio
Copy link
Owner

ericgio commented Mar 31, 2022

Looks like I needed to rm -rf node_modules/ && yarn install for .yarnclean to work. Hopefully that did the trick.

@DASPRiD
Copy link
Contributor Author

DASPRiD commented Mar 31, 2022

@ericgio That was it, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants