Skip to content

Conversation

@aarongarciah
Copy link
Member

@aarongarciah aarongarciah commented Oct 8, 2024

Extracted from #42824 to fix the following TS error in the Timeline component when using React 19 types:

image

@aarongarciah aarongarciah added typescript package: lab Specific to the lab. scope: timeline Changes related to the timeline. integration: react About improving React support. labels Oct 8, 2024
@mui-bot
Copy link

mui-bot commented Oct 8, 2024

Netlify deploy preview

https://deploy-preview-44043--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 7b190cd

@aarongarciah aarongarciah changed the title [lab] Fix Timeline types for React 19 [lab][Timeline] Fix types for React 19 Oct 8, 2024
@aarongarciah aarongarciah marked this pull request as ready for review October 8, 2024 14:57
@aarongarciah aarongarciah requested a review from Janpot October 8, 2024 14:59
Comment on lines -24 to -25
const TimelineRoot = styled('ul' as const, {
name: 'MuiTimeline' as const,
Copy link
Member Author

Choose a reason for hiding this comment

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

I removed these since they look unnecessary.

@aarongarciah aarongarciah merged commit d6e5ade into mui:master Oct 8, 2024
22 checks passed
@aarongarciah aarongarciah deleted the timeline-react-19-types branch October 8, 2024 15:17
DiegoAndai pushed a commit to DiegoAndai/material-ui that referenced this pull request Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration: react About improving React support. package: lab Specific to the lab. scope: timeline Changes related to the timeline. typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants