[wp-trac] [WordPress Trac] #53398: Twenty Twenty-One: Polish new blocks added in 5.8

[WordPress Trac]

#53398: Twenty Twenty-One: Polish new blocks added in 5.8
---------------------------------------+---------------------
 Reporter:  desrosj                    |       Owner:  (none)
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  5.8
Component:  Bundled Theme              |     Version:  trunk
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:
---------------------------------------+---------------------

Comment (by ryelle):

 > Scenario 1: The text has no margin on the sides.

 This seems to happen with full-width headings too, so I think we could
 consider this “expected”? There’s still no spacing around the sides with a
 background color, which is not the same as other blocks. I've attached a
 PR which just adds a little space for Query Loops with background colors.

 > Scenario 1b: You’ll notice that there is a margin appropriately when
 starting with the grid view.

 I think @joen’s patch from #53389 fixes this issue.

 > Last scenario: Full width query loop is not full width on frontend.

 I’m able to replicate this, and it seems to be that the columns block in
 this pattern is set to wide width, so it maxes out at wide even if the
 container Query Loop is set to full width. I don't know what the best
 solution would be here - maybe remove the alignment on those columns, and
 add it to the Query Loop?

 Steps to replicate this one:
 - Add a Query Loop block
 - Click the grid icon to view all patterns
 - Click "Image at left"
 - Make the Query Loop full width

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53398#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform