-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
DaisyUI v5 Theme for React JSON Schema Form v6 #4551
base: rjsf-v6
Are you sure you want to change the base?
Conversation
Redesigned form templates with connected card styling, consistent spacing, and improved layout. Added proper visual hierarchy for nested elements, optimized control button positioning, and implemented cross-browser date/time widgets. Applied DaisyUI utility classes consistently for a cohesive interface while maintaining full RJSF functionality.
packages/daisyui/src/widgets/CheckboxesWidget/CheckboxesWidget.tsx
Outdated
Show resolved
Hide resolved
PR feedback. PR review
Looking good.
|
@heath-freenome - ready, sir. |
packages/daisyui/src/templates/ArrayFieldDescriptionTemplate/ArrayFieldDescriptionTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/ArrayFieldTemplate/ArrayFieldTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/WrapIfAdditionalTemplate/WrapIfAdditionalTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/WrapIfAdditionalTemplate/WrapIfAdditionalTemplate.tsx
Outdated
Show resolved
Hide resolved
@heath-freenome I think we're closer now.
Core Rating Widget![]() Default Style![]() Daisy Style ❤️![]() |
packages/daisyui/src/templates/ArrayFieldDescriptionTemplate/ArrayFieldDescriptionTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/ArrayFieldDescriptionTemplate/ArrayFieldDescriptionTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/ArrayFieldTemplate/ArrayFieldTemplate.tsx
Outdated
Show resolved
Hide resolved
@inchoate you have a build error to fix with linting. Consider running |
Fixes selects. Uses Fieldsets instead of custom arrays. Uses join for buttons when appropriate. Calendar fix
packages/daisyui/src/templates/ArrayFieldTemplate/ArrayFieldTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/FieldHelpTemplate/FieldHelpTemplate.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/src/templates/DescriptionField/DescriptionField.tsx
Outdated
Show resolved
Hide resolved
packages/daisyui/tailwind.config.js
Outdated
@@ -0,0 +1,20 @@ | |||
/** @type {import('tailwindcss').Config} */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I'm guessing this is working without the conversion
* Updated .github templates to add: fluent-rc, shadcn, daisy * Addresses all PR comments * Updates README
const onCopyClick = useMemo(() => onCopyIndexClick(index), [index, onCopyIndexClick]); | ||
const onRemoveClick = useMemo(() => onDropIndexClick(index), [index, onDropIndexClick]); | ||
const onArrowUpClick = useMemo(() => onReorderClick(index, index - 1), [index, onReorderClick]); | ||
const onArrowDownClick = useMemo(() => onReorderClick(index, index + 1), [index, onReorderClick]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could switch these to useCallback()
instead
Integrate DaisyUI v5 with RJSF v6
This PR integrates DaisyUI v5 with React JSON Schema Form v6, providing a complete theme implementation with enhanced visual styling and component architecture.
Changes
nested.ts
to enhance the demoTesting
Tested across multiple browsers with complex nested schemas including:
Screenshots
@heath-freenome @saadeghi