web-illinois / illinois_framework_theme

An Illinois branded Drupal theme based off of the bootstrap4 theme.
6 stars 1 forks source link

Accordion - add component #561

Closed mwllr closed 2 years ago

mwllr commented 2 years ago

Need to add accordion as an option for paragraph type. Need the ability to add text above and below the accordion in the paragraph.

Fields: Heading - h2 Text Box - so we can put a paragraph or more before the accordion. Accordion section

Text Box - so we can put a paragraph or more after the accordion.

https://github.com/web-illinois/toolkit/wiki/il-accordion

github-actions[bot] commented 2 years ago

Branch trubach/issue-561-accordion-add-component created!

trubach commented 2 years ago

@mwllr - How strongly do you feel that we need the options for opening single/multiple accordions at a time, along with sold/bordered display? I'm inclined to keep the editing form simple, and set it to multiple allowed open, and using the solid display.

Then, in theory, someone could override the templates we have in our theme to change the behavior of the accordions. They'd need to add their own TWIG files, however.

What do you think? I have a working solution on my localdev currently, minus the multiple/single and solid/bordered options.

trubach commented 2 years ago

I talked with Melissa and she's OK with merging what's currently developed. I will create a new ticket to add the solid/bordered and multiple open options.

mwllr commented 2 years ago

yep. Melissa will take basic functionality now and upgrades later ;)

mwllr commented 2 years ago

Accordion paragraph type added and functions properly. https://illinois-drupal.web.illinois.edu/paragraphs/accordion