Skip to content

Conversation

@oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Nov 30, 2025

A quick follow-up on #47165, while I landed on the page by chance, looking at the base-ui.com backlinks. Having allowWheelScrub seems pointless:

  1. This is an option from Base UI. What we build on top of it should assume that Base UI already has the correct default behaviors. So, unless there is a plan to change it in Base UI, but it can't change now because of BCs, I believe we need to rely on Base UI default behaviors, not hide them.
  2. It's listed as part of why a component like this one can bring value: [number field] Add Number Input component #19154, in https://mui.com/material-ui/react-text-field/#type-quot-number-quot too

Preview: https://deploy-preview-47397--material-ui.netlify.app/material-ui/react-number-field/

@oliviertassinari oliviertassinari added type: bug It doesn't behave as expected. scope: number field Changes related to the number field. labels Nov 30, 2025
@mui-bot
Copy link

mui-bot commented Nov 30, 2025

Netlify deploy preview

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

Bundle size report

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against c42539d

@siriwatknp siriwatknp merged commit 6b69cd6 into mui:master Dec 1, 2025
23 of 24 checks passed
@oliviertassinari oliviertassinari deleted the number-input-fix-scroll branch December 1, 2025 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: number field Changes related to the number field. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants