-
Notifications
You must be signed in to change notification settings - Fork 8
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
Remove volto-subblocks #51
Comments
@pnicolli My vote would go to use the ObjectListWidget. We're using it extensively everywhere, it's a good fit. |
@pnicolli The ObjectListWidget supports a schemaExtender (which predates Volto's schemaEnhancers), so the framework is capable of holding the different fields as a single object list field. https://github.com/plone/volto/blob/eef928fd8be30d0a66c1fdb923bb43ec9757e10a/src/components/manage/Widgets/ObjectListWidget.jsx#L237-L239 Also https://6.dev-docs.plone.org/storybook/?path=/story/edit-widgets-object-list-json--schema-extender |
My vote would be for the 2nd option, using the existing blocks editor in the long term, but I appreciate how much more work this would be compared to using the ObjectListWidget. I personally find the current 'Add field' UI a little cumbersome when working on longer forms and I don't think moving that 'add' button into the sidebar would speed things up much. Additionally, having the flexibility to insert custom blocks in the middle of a form (replacing the |
@JeffersonBledsoe You're right, I misread @pnicolli comment regarding second option. This would move the addon in the direction of kitconcept's design of the forms block. Cool! |
This is meant to be a discussion topic. At this time, there is no deadline for this change, but you never know what the future will bring.
volto-subblocks
is an old implementation for having sortable inner blocks and there are now at least two better ways of doing the same thing.There are two possible ways to achieve removal of
volto-subblocks
.ObjectListWidget
from core voltoWhile the second implementation is probably more easily extensible and maybe future proof, the first one is way easier and quicker to implement. I would probably still go with the first one, since
volto-subblocks
really needs to go, but this is open to feedback and discussion.The text was updated successfully, but these errors were encountered: