Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.4k stars 1.98k forks source link

CoBlocks Masonry block loses grid layout when added to template on Simple Sites #80627

Open kingsleyinfo opened 1 year ago

kingsleyinfo commented 1 year ago

Quick summary

The masonry block loses its alignment and gets stacked when used in a template ~part/ template pattern that's applied to a page with a slideshow block.~


kayshot-2023-08-15-12 50 30

To explain how this occurs, see the reproduction steps below.

The workaround I discovered was to add a blank image block below the slider block while adding content to the page.

Steps to reproduce

  1. Apply the Hari theme.
  2. Edit the page template and include a template pattern with masonry styled gallery block.
  3. Add this pattern to the footer or under the post content block.
  4. Create a new page and include a slider block.
  5. Apply the page template you edited above.
  6. Save changes and preview the page.

Video: https://github.com/Automattic/wp-calypso/assets/27249804/6e1c4929-715e-4a0a-b4bb-ad39be504141

What you expected to happen

I expected the Masonry block to display properly irrespective of which block was used in the page

What actually happened

Masonry blocks are stacked vertically when a slideshow block is used on the main page.

Impact

One

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

Simple

Logs or notes

Not tested on Atomic sites yet.

Tested with Hari theme Test page: https://chpreloaded.blog/multi-column/

Ticket: 6667824-zd-a8c

github-actions[bot] commented 1 year ago

Support References

This comment is automatically generated. Please do not edit it.

cuemarie commented 1 year ago

📌 SCRUBBING : RESULT - Replicated / Could Not Replicate / Uncertain

📌 FINDINGS/SCREENSHOTS/VIDEO

Screenshots/Recordings

Simple site test

Masonry Block added to Post

Markup on 2023-08-16 at 10:28:19

Masonry Block added to Template

Markup on 2023-08-16 at 10:12:19

It seems when the Masonry Block is in the template, and there's not also a Masonry block on the post itself, the Masonry block no longer contains

Atomic site test - not replicable

CoBlocks Version 3.1.2

https://github.com/Automattic/wp-calypso/assets/27249804/6bc34c65-69be-49b5-a14f-bbae91b47544

📌 ACTIONS

📌 Message to Author

mdtanjid0 commented 3 weeks ago

Got a similar issue here: 8650624-zen advised them to use Tiled Gallery Block.