Skip to content

[Popover] Remove transition onX props#22184

Merged
mbrookes merged 4 commits into
mui:nextfrom
mbrookes:popover-remove-transition-onX-props
Aug 14, 2020
Merged

[Popover] Remove transition onX props#22184
mbrookes merged 4 commits into
mui:nextfrom
mbrookes:popover-remove-transition-onX-props

Conversation

@mbrookes

@mbrookes mbrookes commented Aug 13, 2020 •
edited by oliviertassinari
Loading

Copy link
Copy Markdown
Member

Breaking change

The onE* transition props were removed. Use TransitionProps instead.

<Popover
-  onEnter={onEnter}
-  onEntered={onEntered},
-  onEntering={onEntered},
-  onExit={onEntered},
-  onExited={onEntered},
-  onExiting={onEntered}
+  TransitionProps={{
+    onEnter,
+    onEntered,
+    onEntering,
+    onExit,
+    onExited,
+    onExiting,
+  }}
/>

#17047 (comment)

@mbrookes mbrookes added breaking change Introduces changes that are not backward compatible. component: Popover The React component. labels Aug 13, 2020
@mbrookes mbrookes added this to the v5 milestone Aug 13, 2020
oliviertassinari
oliviertassinari approved these changes Aug 13, 2020
View reviewed changes
Comment thread docs/src/pages/guides/migration-v4/migration-v4.md Outdated
@mui-pr-bot

mui-pr-bot commented Aug 13, 2020 •
edited
Loading

Copy link
Copy Markdown

Details of bundle changes

Generated by 🚫 dangerJS against cba8298

@github-actions github-actions Bot mentioned this pull request Aug 13, 2020
Closed
@mbrookes
mbrookes merged commit 183f267 into mui:next Aug 14, 2020
@mbrookes
mbrookes deleted the popover-remove-transition-onX-props branch August 14, 2020 07:48
@mbrookes mbrookes mentioned this pull request Aug 14, 2020
Merged
@oliviertassinari oliviertassinari mentioned this pull request Sep 15, 2020
Closed
42 tasks
@mbrookes mbrookes mentioned this pull request Sep 23, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@oliviertassinari oliviertassinari oliviertassinari approved these changes

Assignees

No one assigned

Labels

breaking change Introduces changes that are not backward compatible. component: Popover The React component.

Projects

None yet

Milestone

v5.0.0-beta.0

Development

Successfully merging this pull request may close these issues.

3 participants

@mbrookes @mui-pr-bot @oliviertassinari