[Accordion] Does prop padding make sense?

[GomezIvann]

My approach is the same as for tokens in this situation, should we have props that modify or affect the custom content of container components, such as this one or Card, or Dialog, etc? I don't think this prop give too much value and opens the door to ask why not other properties such as margins, alignment, etc.

This can also be applied to Card's prop contentPadding.

[aweell]

I don't think we're adding so much value with this prop other than providing a set of pre-defined values since it's as easy as using a container with custom styles wrapping the content:

I would understand the case when we allow changing the padding of the elements we provide by default (e.g. change the padding of the accordion heading container, which affects the label, helper text, and icon)

Frankly speaking, I wouldn't miss it if it's not there since we're developing other ways to apply padding, and component spacing in our layout proposal...

[GomezIvann]

I totally agree with you Alex, I don't think we are giving too much value with this prop, since it can be done in a few lines and is much more customizable.

[GomezIvann]

Since the release of the Layout components, we have decided here that removing padding props is the first step of the new strategy about how we want users to create layouts and arrange items on the screen.