Automattic / jetpack

Security, performance, marketing, and design tools — Jetpack is made by WordPress experts to make WP sites safer and faster, and help you grow your traffic.
https://jetpack.com/
Other
1.59k stars 799 forks source link

Slideshow block: absolute positioning breaks layout #13234

Open goblinartificer opened 5 years ago

goblinartificer commented 5 years ago

I am not sure if this is purely a theme issue, but I thought I'd report it both places just in case there's something that could be tidied on this front. Feel free to close if it's nothing to do with this block itself!

Steps to reproduce the issue

  1. Activate Christopher theme on a WordPress.com site.
  2. Create a home page with some paragraph/heading blocks and a slideshow block. It doesn't matter if the other blocks are before or after the slideshow.
  3. Set this page's page template to either Default or Front Page.
  4. View site.

What I expected

A slideshow and some paragraphs.

What happened instead

A very large blank space, and paragraphs running in a single line off the page to the right.

More information This can be seen live on limitedepsilon.wordpress.com at the moment. In general, setting the slideshow block's width to anything wider than the content container will make the paragraphs match its width.

Both these front page templates on this theme have a div with the ID masonry-container, and the article element uses absolute positioning. This causes this JS error:

First discussed in p1565567548017400-slack-tickets

stale[bot] commented 4 years ago

This issue has been marked as stale. This happened because:

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.