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

[base-ui][TextareaAutosize] Add border-box to demo #40646

Merged

Conversation

ANUGLYPLUGIN
Copy link
Contributor

@ANUGLYPLUGIN ANUGLYPLUGIN commented Jan 17, 2024

The bug mui/base-ui#68 seems to have relationship with #40634 due to customer root has no border-box attribute.

@mui-bot
Copy link

mui-bot commented Jan 17, 2024

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 64e7279

@ANUGLYPLUGIN
Copy link
Contributor Author

I don't quite understand why execFileSync in test-unit gets wrong. Anybody has ideas about that?

@michaldudak
Copy link
Member

It times out sometimes. It's not related to your change. I restarted the job.

@danilo-leal danilo-leal added the docs Improvements or additions to the documentation label Jan 17, 2024
@ANUGLYPLUGIN
Copy link
Contributor Author

It times out sometimes. It's not related to your change. I restarted the job.

Oh this is indeed the case, I failed ten times in local. Anyway thanks

Copy link
Contributor

@zanivan zanivan left a comment

Choose a reason for hiding this comment

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

Thanks for tackling this! 🚀

@zanivan zanivan merged commit c5bc62a into mui:master Jan 19, 2024
22 checks passed
@ANUGLYPLUGIN ANUGLYPLUGIN deleted the Young/base-ui-textarea-autoresize-demo branch January 19, 2024 18:00
@ANUGLYPLUGIN
Copy link
Contributor Author

Thanks for tackling this! 🚀

Thanks for reviewing my first pr!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: TextareaAutosize The React component. docs Improvements or additions to the documentation package: base-ui Specific to @mui/base
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants