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

Margin toevoegen met de nieuwe rich-text component #793

Closed
3 of 4 tasks
AliKdhim87 opened this issue Jun 28, 2024 · 4 comments · Fixed by #851
Closed
3 of 4 tasks

Margin toevoegen met de nieuwe rich-text component #793

AliKdhim87 opened this issue Jun 28, 2024 · 4 comments · Fixed by #851
Assignees

Comments

@AliKdhim87
Copy link
Contributor

AliKdhim87 commented Jun 28, 2024

Beschrijving:
In de accordion klopt de spacing niet met de Figma van Utrecht (@AliKdhim87 kun je hier een link naar die Figma frame toevoegen?)

Door spacing toe te voegen met de rich-text component wordt dat misschien al opgelost.

  • Voeg rich-text toe
  • Test of de accordion nu klopt met Figma
  • Maak notities van de verschillen die je nog ziet in een comment in dit issue
  • Laat oplossing door Utrecht reviewen
@Yolijn Yolijn changed the title Margin toevoegen aan de Accordeon-component Margin toevoegen met de nieuwe rich-text component Aug 16, 2024
@Yolijn Yolijn moved this to Ready in Utrecht Strapi CMS Aug 16, 2024
@AliKdhim87
Copy link
Contributor Author

Figma link

@scar055
Copy link
Contributor

scar055 commented Aug 20, 2024

De content in de accordions worden nu goed uit gelijned voor utrecht pdc.

Image

Nu alleen de gap tussen de uitklap button en de accordion content weg halen.
Als de onderste accordion dicht is staat de accordion goed volgens de margins.

@scar055
Copy link
Contributor

scar055 commented Aug 20, 2024

Utrecht vth staat de content in accordion goed uitgelijned.

Image

Op de vth is ook de gap tussen de button en de content.
Als de onderste accordion dicht is staat de accordion goed volgens de margins

En in het storybook is de gap tussen de button en content ook het zelfde.

@scar055 scar055 moved this from In progress to Ready in Utrecht Strapi CMS Aug 20, 2024
@scar055
Copy link
Contributor

scar055 commented Aug 22, 2024

deze issue kan ik verder werken als deze issue klaar is https://github.com/orgs/frameless/projects/45/views/8?pane=issue&itemId=75023906

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

Successfully merging a pull request may close this issue.

2 participants