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

Reusable Blocks: Convert to Regular Block does not preserve nested blocks #9278

Closed
ZebulanStanphill opened this issue Aug 23, 2018 · 6 comments
Assignees
Labels
[Feature] Nested / Inner Blocks Anything related to the experience of nested/inner blocks inside a larger container, like Group or P [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Type] Bug An existing feature does not function as intended

Comments

@ZebulanStanphill
Copy link
Member

ZebulanStanphill commented Aug 23, 2018

The issue

If you save a block containing nested blocks as a Reusable Block, then it works just fine as a Reusable Block. But if you try to convert an instance of that Reusable Block to a regular block, then the resulting regular block will not contain any of the nested blocks that the Reusable Block had.

Steps to reproduce

  1. Insert a Columns block.
  2. Insert some other block into the Columns block.
  3. Open the ellipsis (More Options) menu and choose Add to Reusable Blocks.
  4. Finish naming and saving the Reusable Block.
  5. Notice that the nested blocks are saved in the Reusable Block as expected.
  6. Open the ellipsis menu again and choose Convert to Regular Block.
  7. Notice that the nested blocks are gone.

Expected behavior

Reusable blocks should preserve nested blocks when an instance is converted to a regular block.

Related issues and PRs

@designsimply
Copy link
Member

Tested and confirmed using WordPress 4.9.8 and Gutenberg 3.6.2 and noting that in my case the blocks were preserved but the content inside them was not. (39s)

@designsimply designsimply added [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Feature] Nested / Inner Blocks Anything related to the experience of nested/inner blocks inside a larger container, like Group or P [Type] Bug An existing feature does not function as intended labels Aug 23, 2018
@noisysocks
Copy link
Member

Likely the same root cause as #5754.

@asadrahbar
Copy link

Is anyone working on a fix for this? The issue also occurs when using the Atomic Blocks plugin's AB Container block. I would like to create reusable sections (ex: image with heading, paragraph, and button) that clients could drop onto a page and then edit the content, similar to how the Divi library works. The only workaround for now is to copy the HTML for the nested blocks, paste into an HTML block and then Convert to Blocks - obviously not ideal for users who are afraid of code.

@slimmilkduds
Copy link

@asadrahbar with 3.9 you will be able to create reusable blocks containing multiple blocks without a section block, hopefully someone will pick this issue up soon since that feature goes from being amazing to pretty useless with this bug present

@riemerta6
Copy link

I'm also experiencing the same issue :/ Learned the hard way.

@noisysocks
Copy link
Member

Likely the same root cause as #5754.

After looking into this some more, I think that this is a seperate bug caused by not passing referencedBlock.innerBlocks here:

newBlocks = [ createBlock( referencedBlock.name, referencedBlock.attributes ) ];

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Nested / Inner Blocks Anything related to the experience of nested/inner blocks inside a larger container, like Group or P [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

6 participants