Skip to content

[material-ui][Tabs] Fix ScrollbarSize ref being overriden#44593

Merged
DiegoAndai merged 1 commit intomui:masterfrom
DiegoAndai:fix-scrollbarsize-ref
Nov 28, 2024
Merged

[material-ui][Tabs] Fix ScrollbarSize ref being overriden#44593
DiegoAndai merged 1 commit intomui:masterfrom
DiegoAndai:fix-scrollbarsize-ref

Conversation

@DiegoAndai
Copy link
Member

@DiegoAndai DiegoAndai commented Nov 28, 2024

Closes: #41388
The explanation for the fix: #41388 (comment)

ScrollbarSize is an internal component, not exported, and we don't need to provide a ref, so there's no need for ref merging.

Before and after: https://github.com/DiegoAndai/material-ui-issue-41388-repro
Before is the main branch
After is the with-fix branch that uses this PR's build

@DiegoAndai DiegoAndai added scope: tabs Changes related to the tabs. package: material-ui integration: react About improving React support. labels Nov 28, 2024
@DiegoAndai DiegoAndai self-assigned this Nov 28, 2024
@mui-bot
Copy link

mui-bot commented Nov 28, 2024

Netlify deploy preview

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

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against 3d4aa7f

Copy link
Member

@aarongarciah aarongarciah left a comment

Choose a reason for hiding this comment

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

🚀

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. scope: tabs Changes related to the tabs. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[material-ui][Tabs] Scrollable variant fails test when clicking the tab (error reading 'offsetHeight')

4 participants