-
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
Custom description field #214
Conversation
|
||
You can provide your own implementation of the `DescriptionField` base React component for rendering any description. | ||
|
||
|
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.
nit: one empty line too many
LGTM with nits addressed. It's a little sad we can't accept react elements as description, but that's definitely not an option in a JSONschema :) |
Among other cosmetic changes, we now pass custom titles and descriptions using the `fields` property (whereas we were passing it directly before, a behaviour now deprecated).
I've updated the branch with commits addressing your comments :) |
Perfect, thanks o/ |
Released as part of v0.31.0. |
Awesome, thanks! |
This adds the ability to customize the description using a description field.
This breaks the previous behaviour: it was returning a
p
for ArrayFields anddivs
for ObjectFields. This is now returning adiv
for both.I also added some tests in the ArrayField to check that it's able to handle custom TitleFields.
paired with @Natim, r? @n1k0