Update Query Loop patterns with less posts per page

[ntsekouras]

I think we need to 'lighten' the Query Loop patterns regarding the number of posts it previews. The point of a pattern is to showcase some design and with some pre-selected blocks(the inner Post blocks, etc..). It's already opinionated about the number of posts, so we could just show the minimum number required to make the pattern show its point.

We need to find a good way to handle patterns performance in core, but for 5.9 is the least we can do.

The Gutenberg PR for this change is here: ​https://github.com/WordPress/gutenberg/pull/37475

Similar PR for TwentyTwentyTwo theme is here: ​https://github.com/WordPress/twentytwentytwo/pull/304

Trac ticket: https://core.trac.wordpress.org/ticket/54639

I think we need to 'lighten' the Query Loop patterns regarding the number of posts it previews. The point of a pattern is to showcase some design and with some pre-selected blocks(the inner Post blocks, etc..). It's already opinionated about the number of posts, so we could just show the minimum number required to make the pattern show its point.

[SergeyBiryukov]

Thanks for the ticket! Moving to 5.9 for visibility.

Thanks for reviewing @SergeyBiryukov , but let's wait just a bit before merging as there were some concerns raised about this by @mtias here: ​https://github.com/WordPress/gutenberg/pull/37475#issuecomment-996534835.

There is the possibility of reverting the GB PR and that would make this PR obsolete.

^{I was hoping that my review at ​https://github.com/WordPress/wordpress-develop/pull/2063#pullrequestreview-835446418 would add a red "changes requested" mention by the merge button. It apparently didn't, so merger beware. :)}

[hellofromTonya]

​Gutenberg's PR 37475 has been merged into Gutenberg's trunk and is marked for backporting. Will monitor to ensure it's part of the Beta 4 resync updates patch that @isabel_brison will be doing for Beta 4.

The ​TT2 PR however was not part of the resync for Beta 4. cc @jffng @kjellr Can this PR be ready and backported for Beta 4?

[jffng]

Looking at the TT2 PR, it looks there is still some discussion to be had before merging. Will take another look but if we have to reduce the number of posts per page, I'd prefer to get @kjellr to take another look at each pattern and determine what is the right number of posts to show for each Query pattern while still providing a good preview experience.

[hellofromTonya]

Replying to jffng:

Looking at the TT2 PR, it looks there is still some discussion to be had before merging. Will take another look but if we have to reduce the number of posts per page, I'd prefer to get @kjellr to take another look at each pattern and determine what is the right number of posts to show for each Query pattern while still providing a good preview experience.

Thanks @jffng. I just realized, DOH, that this patch is on hold and may be reverted from the Gutenberg plugin. Waiting on a decision from Matias.

---

NOTE FOR CORE COMMITTERS:
The patch in this ticket is on hold pending a decision from Matias.

I'm closing this as we reverted the GB PR eventually.

[hellofromTonya]

Closing this ticket as ​https://github.com/WordPress/gutenberg/pull/37475 was reverted.