-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[Popover] Deprecate transition onX props #22202
[Popover] Deprecate transition onX props #22202
Conversation
893512e
to
edb369a
Compare
Details of bundle changes.Comparing: 07943fb...9d64fe3 Details of page changes
|
'Popover', | ||
); | ||
|
||
// expect(console.error.callCount).to.equal(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess there are other prop types warnings beside the deprecation warning? The Popover might have required props. Saw this in another PR of you as well so this might be the same issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Popover might have required props.
That makes sense, thanks.
Saw this in another PR of [yours] as well
Yeah, copy-pasta. Fixed, thanks.
This reverts commit c360619.
Popover
The onE* transition props are deprecated. Use TransitionProps instead.
Ref #22184