Skip to content

Fix 4707 - Fixed additional properties for daisyui theme #4722

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

Merged
merged 2 commits into from
Aug 18, 2025

Conversation

heath-freenome
Copy link
Member

Reasons for making this change

Fixes #4707 by rendering the WrapIfAdditionalTemplate inside of the FieldTemplate

  • Updated the WrapIfAdditionalTemplate to properly render the delete button and key input around the children when the schema element is an additional property
  • Updated the FieldTemplate to properly wrap the Field with the WrapIfAdditionalTemplate
  • Updated the snapshots accordingly
  • Updated the CHANGELOG.md accordingly

Before:
Screenshot 2025-08-15 at 1 29 54 PM

After:
Screenshot 2025-08-15 at 1 23 29 PM

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests. I've run npx nx run-many --target=build --exclude=@rjsf/docs && npm run test:update to update snapshots, if needed.
    • I've updated docs if needed
    • I've updated the changelog with a description of the PR
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

Fixes rjsf-team#4707 by rendering the `WrapIfAdditionalTemplate` inside of the `FieldTemplate`
- Updated the `WrapIfAdditionalTemplate` to properly render the delete button and key input around the `children` when the schema element is an additional property
- Updated the `FieldTemplate` to properly wrap the Field with the `WrapIfAdditionalTemplate`
- Updated the snapshots accordingly
- Updated the `CHANGELOG.md` accordingly
@heath-freenome heath-freenome merged commit 0748a0c into rjsf-team:main Aug 18, 2025
4 checks passed
@heath-freenome heath-freenome deleted the fix-4707 branch August 18, 2025 16:35
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.

AdditionalProperties are not supported in DaisyUI template
2 participants