-
-
Notifications
You must be signed in to change notification settings - Fork 32.7k
Closed
Labels
scope: systemThe system, the design tokens / styling foundations used across components. eg. @mui/system with MUIThe system, the design tokens / styling foundations used across components. eg. @mui/system with MUItype: new featureExpand the scope of the product to solve a new problem.Expand the scope of the product to solve a new problem.
Description
- I have searched the issues of this repository and believe that this is not a duplicate.
Summary 💡
Currently the borderRadius property on the Box component matches the theme.shape values. So for defining a borderRadius, usually we have the following code:
<Box borderRadius="borderRadius" />The proposal (taken from #23053 (comment)) is to consider the theme.shape.borderRadius as a value used for multiplying the values inside the borderRadius.
Examples 🌈
Here are a few examples of how the new conversion could be used
theme = createMuiTheme({ shape: { borderRadius: 2}});
// ...
<Box borderRadius={1} /> // borderRadius: 2
<Box borderRadius={3} /> // borderRadius: 6
<Box borderRadius={10} /> // borderRadius: 20Motivation 🔦
With this, we will have a more flexible approach for defining the borderRadius values and it would work consistently with the spacing values already available in the system.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
scope: systemThe system, the design tokens / styling foundations used across components. eg. @mui/system with MUIThe system, the design tokens / styling foundations used across components. eg. @mui/system with MUItype: new featureExpand the scope of the product to solve a new problem.Expand the scope of the product to solve a new problem.