Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Details component addition #581

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Conversation

savitris
Copy link
Contributor

@savitris savitris commented Apr 4, 2024

Dit is een Proof of Concept van het gebruik van een Details component om een Accordion component te vervangen.

De Accordions van elk pagina van de VTH pagina is vervangen om het makkelijker te maken het verschil te zien.
Onder elk Detail component ligt een Accordion component om ze makkelijker te kunnen vergelijken.

Om dit zelf te testen, start VTH op en zorgt dat je op een pagina bent waar je een Uitklapper aan hebt toegevoegd in Strapi.
Er moet heel veel content in een Uitklapper panel, (de content moet langer zijn dan de hoogte van je scherm), zodat als je met je keyboard navigeert, je het verschil goed kan zien (probeer tussen de panels door te tabben en in elk panel de informatie door te lezen door het gebruik van de Down Arrow toets).

Screenshot of webpage with an accordion and a details element Screenshot of the Strapi interface for adding accordions to a page

@Robbert Robbert marked this pull request as draft April 4, 2024 15:29
@scar055
Copy link
Contributor

scar055 commented Apr 4, 2024

ik recommend dat je de commits die ik heb gemaakt dropped die zijn gefixed al op main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants