Skip to content
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

Open
pnicolli opened this issue Nov 13, 2022 · 4 comments
Open

Remove volto-subblocks #51

pnicolli opened this issue Nov 13, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@pnicolli
Copy link
Member

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.

  1. using the standard ObjectListWidget from core volto
  2. using an inner blocks editor, therefore changing all widgets to restricted blocks

While 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.

@pnicolli pnicolli added the enhancement New feature or request label Nov 13, 2022
@tiberiuichim
Copy link
Member

@pnicolli My vote would go to use the ObjectListWidget. We're using it extensively everywhere, it's a good fit.

@JeffersonBledsoe
Copy link
Member

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 static text field type that we currently have with volto-slate/ draft) seems to fit within the blocks-focused workflow that Volto applies so well. More than happy to go with option 1 and maybe look at option 2 as another evolution after the ObjectListWidget :)

@tiberiuichim
Copy link
Member

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants