Skip to content

docs: operate: common props for UI widgets #832

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

palkerecsenyi
Copy link
Member

Closes inveniosoftware/react-invenio-forms#297

Description

  • As part of overridable: Implement for custom fields react-invenio-forms#298, UI widgets are being given a standardised common set of props (on top of which each widget can have additional, more specific props)

  • This PR changes the reference docs for custom field widgets by splitting into the two distinct sections of "common props" and "widget-specific props"

  • Some of the prop names have been updated to stay consistent with the names we use for the built-in deposit form fields (e.g. description -> helpText) so this has been reflected in the docs.

* As part of
inveniosoftware/react-invenio-forms#298, UI
widgets are being given a standardised common set of props (on top of
which each widget can have additional, more specific props)

* This PR changes the reference docs for custom field widgets by
splitting into the two distinct sections of "common props" and
"widget-specific props"
Copy link
Collaborator

@fenekku fenekku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Since we are preparing to release v13.0, I just want to make sure:

  • this is only applicable if inveniosoftware/react-invenio-forms#298 gets merged
  • and that's likely to happen only after v13.0 release, so we should hold on merging until that's finalized
  • It seems like something nice to have for a v13.1 :)

* Made doc headings clearer for custom field widgets by only showing
props that are additional to the common props for each widget
@fenekku
Copy link
Collaborator

fenekku commented Jul 18, 2025

With respect to when to get this in: we are continuing the discussion with a couple of maintainers about how to arrange version releases and pins next week. So I guess we are still waiting to determine when that would all go in 😁 . We'll keep an eye on it and try to get things moving in a timely fashion though!

@palkerecsenyi palkerecsenyi marked this pull request as ready for review July 25, 2025 13:45
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.

custom fields: Add Overridable support
3 participants