search

indiv-legacy / kit

https://indivorg.github.io/kit
Apache License 2.0
0 stars 0 forks source link

feat: Accordion #25

Closed simenandre closed 2 years ago

simenandre commented 2 years ago

Adds a nice accordion

Tasks

braaar commented 2 years ago

First impressions: I like the look, but I wonder whether we would want som option for removing the separator line. There might be a scenario where one would want to contrast the header and the content with differently coloured backgrounds

simenandre commented 2 years ago

First impressions: I like the look, but I wonder whether we would want som option for removing the separator line. There might be a scenario where one would want to contrast the header and the content with differently coloured backgrounds

Ah! You're right, we should probably not include it at all?

netlify[bot] commented 2 years ago

✔️ Deploy Preview for indiv-kit ready!

🔨 Explore the source changes: 02ea033ffd3a363c7d83cfd80097f4c781167a64

🔍 Inspect the deploy log: https://app.netlify.com/sites/indiv-kit/deploys/619d585f90f99f00088b06dc

😎 Browse the preview: https://deploy-preview-25--indiv-kit.netlify.app

braaar commented 2 years ago

What happened to the arrow?

simenandre commented 2 years ago

What happened to the arrow?

It died. At least for now 😅

braaar commented 2 years ago

Screenshot 2021-11-25 at 9 41 03

Hard corners when filling boxes with colour. Shouldn't the accordion enforce rounded corners?