-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[Accordion] Horizontal padding #20580
Comments
@esseswann The Material Design specification does no longer covers the expansion panel component. This gives us more freedom around the actual look. Regarding your request, I have looked at Bootstrap, they use the style of the Card Header. So in our case, I think that it would make sense to do the same and to move forward with your request. diff --git a/packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js b/packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
index 7ed48db1a..b61281403 100644
--- a/packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
+++ b/packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
@@ -17,7 +17,7 @@ export const styles = (theme) => {
display: 'flex',
minHeight: 8 * 6,
transition: theme.transitions.create(['min-height', 'background-color'], transition),
- padding: theme.spacing(0, 3),
+ padding: theme.spacing(0, 2),
'&:hover:not($disabled)': {
cursor: 'pointer',
}, Do you want to submit a pull request? :) On a side note, I think that we should rename the component in v5
|
Can do, should it be from some v5 branch or 4.x? |
We don't have a v5 branch yet. master is perfect. |
Current Behavior 😯
Expansion panel has 24px padding, while list items have 16px. I am not sure if it corresponds to the spec, but it makes you add extra elements for unification
Expected Behavior 🤔
Elements should have unified horizontal padding
Your Environment 🌎
The text was updated successfully, but these errors were encountered: