Skip to content

[docs][system] Update sizing docs to clarify (0, 1] behavior.#47845

Merged
ZeeshanTamboli merged 1 commit intomui:masterfrom
matthias-ccri:patch-1
Feb 24, 2026
Merged

[docs][system] Update sizing docs to clarify (0, 1] behavior.#47845
ZeeshanTamboli merged 1 commit intomui:masterfrom
matthias-ccri:patch-1

Conversation

@matthias-ccri
Copy link
Contributor

Adding some qualifiers to the docs that said "[All] numbers are treated as pixel values" -- that's simply not the case. For library authors animating values or supporting continuous values that may go into the range of 0..1 -- they may encounter surprises if they take at face value the statement "numbers are pixels" (if only that were true!) 😄

RE the phrasing change ("converted to" versus "treated as") -- no conversion is taking place. Conversion happens when converting a number from one unit into another. It requires a "from" unit and a "to" unit. It appears that "pixels" is the "from" unit. And pixels is also the "to" unit because the style output is a pixel string like ${n}px. One does not say "convert a pixel to a pixel".

The phrasing of "treated as" makes sense. The library treats these numbers as if they were pixel values. Hence the phrase "treated as numbers".

Signed-off-by: Matthias <18034092+matthias-ccri@users.noreply.github.com>
@mui-bot
Copy link

mui-bot commented Feb 24, 2026

Netlify deploy preview

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 f0485fe

@ZeeshanTamboli ZeeshanTamboli added docs Improvements or additions to the documentation. labels Feb 24, 2026
@ZeeshanTamboli ZeeshanTamboli changed the title Update sizing.md to clarify (0, 1] behavior. [docs][system] Update sizing docs to clarify (0, 1] behavior. Feb 24, 2026
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

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

Makes sense. Thanks!

@ZeeshanTamboli ZeeshanTamboli added scope: system The system, the design tokens / styling foundations used across components. eg. @mui/system with MUI v7.x needs cherry-pick The PR should be cherry-picked to master after merge. labels Feb 24, 2026
@ZeeshanTamboli ZeeshanTamboli merged commit 0081c48 into mui:master Feb 24, 2026
23 of 24 checks passed
@github-actions
Copy link

Cherry-pick PRs will be created targeting branches: v7.x

github-actions bot pushed a commit that referenced this pull request Feb 24, 2026
Signed-off-by: Matthias <18034092+matthias-ccri@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. needs cherry-pick The PR should be cherry-picked to master after merge. scope: system The system, the design tokens / styling foundations used across components. eg. @mui/system with MUI v7.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants