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

feat(tailwind-components): form modal #4814

Draft
wants to merge 32 commits into
base: master
Choose a base branch
from
Draft

Conversation

connoratrug
Copy link
Contributor

What are the main changes you did

  • show form in a modal

  • lots of teaks to modal, changes need to be passed on to current modal users

  • replace hard-coded styles with theme vars

How to test

  • explain here what to do to test this (or point to unit tests)

Checklist

  • updated docs in case of new feature
  • added/updated tests
  • added/updated testplan to include a test for this fix, including ref to bug using # notation

connoratrug and others added 23 commits February 21, 2025 17:07
- add sample page
- add fullpage layout
- update page header to allow title pre and post fix

todo: fix fields component, pull out legend other info parts, fix breadcrumb-styling, add draft badge, add (dummy) action buttons
- add sample page
- add css variable to differentiate a 'normal' title on contast background and a special 'pop' title color
- update the cta cards to use the new pop variant
@connoratrug connoratrug self-assigned this Mar 7, 2025
- move playground header into component
- add form actions ( sticky )
- add form error
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.

1 participant