[Stepper] Remove Paper and built-in padding#22564
Conversation
oliviertassinari
left a comment
There was a problem hiding this comment.
I think that with the change, we can use the default outlined container for the demos, instead of the grey background.
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
oliviertassinari
left a comment
There was a problem hiding this comment.
The mobile stepper still has a white background, it's easier to spot with:
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
|
@oliviertassinari I've tweaked the Button layout and variant for the first demo to better match the original component: Spec: Once it's good, I'll do the rest of the demos. |
|
@mbrookes Looks better 👍 |
Follow 'Stepper Feedback' example at https://material.io/archive/guidelines/components/steppers.html#steppers-types-of-steppers
|
@mbrookes I think that option 1 is better. It will make the source of the demo simpler. Also, I don't think that the stepper should have any padding. |
Oh right, one breaking change at the time, removal of padding could be the very next one 👼.
A couple of thoughts:
|
|
We had a power cut this afternoon, but have been working offline on simplifying the demos. It seems I've already hit on several of the things you suggested. Take a look.
That's down to the demo not having a container. You can't have it both ways. 😄
I think we need to show a functioning Stepper first.
Most of the demos are now simplified.
Already done.
Already simpler, but I'll take another look.
Already done in "Mk 3".
Already done. |







Breaking changes
<div>. Stepper no longer has elevation, nor inherits Paper's props.Closes #18423