Skip to content
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

[Timeline] Migrate TimelineOppositeContent to emotion #25816

Merged

Conversation

vicasas
Copy link
Member

@vicasas vicasas commented Apr 17, 2021

One of #24405

@mui-pr-bot
Copy link

mui-pr-bot commented Apr 17, 2021

Details of bundle changes

@material-ui/lab: parsed: +0.14% , gzip: +0.04%

Generated by 🚫 dangerJS against 21ea343

@oliviertassinari oliviertassinari added the component: timeline This is the name of the generic UI component, not the React module! label Apr 17, 2021
const { classes, className, ...other } = props;
const TimelineOppositeContent = React.forwardRef(function TimelineOppositeContent(inProps, ref) {
const props = useThemeProps({ props: inProps, name: 'MuiTimelineOppositeContent' });
const { className, ...other } = props;

const { align = 'left' } = React.useContext(TimelineContext);
const { classes: contextClasses = {} } = React.useContext(TimelineItemContext);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We won't be needing this after #25822 (review) No preference on the other of how there are merged as long as we release them all at once.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's merge this one and we can actually do the update in #25822 (review)

@mnajdova mnajdova merged commit 419a148 into mui:next Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: timeline This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants