USACE / groundwork

React Components for USACE Applications
https://usace.github.io/groundwork
4 stars 0 forks source link

Accordion header design #44

Open jbkolze opened 2 months ago

jbkolze commented 2 months ago

Some concerns were raised by users about difficulty seeing the delineation between headers / sections of the Accordion component. The current design, by default, is pretty low-contrast without much of a dividing border.

Should the default header design be altered to more clearly define the separation, or should that be left to the user to handle in the Accordion contents using padding, borders, etc.?

willbreitkreutz commented 2 months ago

I'm totally fine with tweaking the default, and we can always give users a back-door to modify as needed

adamscarberry commented 1 month ago

Agree that the dividing line between headers when collapsed is difficult to discern:

image

The border could either be darker (you risk it looking too thick where they meet) or there could be space added between the headers

willbreitkreutz commented 1 month ago

I think a better border makes sense, i'd also like to rip out the border-radius on stacked headers, only add the radius on the first and last.