-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Hidden] Add TypeScript type for className
#20487
Conversation
We have a related discussion on the topic in #19704. |
Details of bundle changes.Comparing: 89f6208...2244c56 Details of page changes
|
@oliviertassinari Ok thanks, so I'll add this specific case in documentation. There are anything else to add ? |
@dolivier1337 HiddenCss and HiddenJs are two private components. The exposed API is the intersection of the capabilities these two components offer. It doesn't really make sense to update the type definitions of HiddenCss as they can't be consumed by users. Let's hold the pull request until we find a path forward in #19704. Thanks for opening this PR. |
698b6cc
to
c6b0304
Compare
c6b0304
to
2244c56
Compare
className
className
@oliviertassinari My bad 🤦♂️, I've just updated this PR to add the It can be a first iteration for the missing type, let me know how can I help you. |
I'm sorry, I'm closing as we will likely want to deprecate this component. Thanks for the care. |
Fixes partially the #19704 issue
Hi,
I want to use the
Hidden
component with a customclassName
with TypeScript 3.8.3.I have the same issue as this comment, the
HiddenCss
component props interface has not been updated.So, I added
className
type within theHiddenCssProps
interface 🙏[UPDATE]
I added
className
type within theHiddenProps
interface and notHiddenCssProps
.