GotJimmy / accordion

Other
47 stars 44 forks source link

Expose the entier properties like borderRadius instead of only a number #22

Closed TheGrinch00 closed 2 years ago

TheGrinch00 commented 2 years ago

Hi, could you please expose for more customization the whole properties?

I want to achieve the feel and look of MUI Accordion but I can't set for the first AccordionSection only top borderRadius, similarly I can't set only bottom radiuses for the last element.

The same is also true for border and many others. Is there a specific reason to why it was made this way?

GotJimmy commented 2 years ago

Hi Antonio, borderRadius was chosen to be a double (instead of BorderRadius) because it is animated (when opening a section watch the borderRadius of the bottom of the header change its value. But you could try to set the headerBorderRadius and contentBorderRadius parameter of just the first and last AccordionSection and that should do the trick for you. Same goes for all the parameters that start with header or content -- you can set them once for all sections when applied to Accordion, or individually when applied to AccordionSection.