psorensen / curated-query-loop Goto Github PK

Hi @psorensen,

We have this use case where we want to be able to set the number of items an editor can manually select.
What do you think of using the Query Loop posts per page setting to define the ContentPicker's maxContentItems?

https://github.com/psorensen/curated-query-loop/blob/main/assets/js/admin/block-extension.js#L30

maxContentItems={query.perPage || 10}

When the user changes the post type setting in the query loop settings, the selectedPosts should be reset, given that the selectedPosts will not be present in the resulting query results.

As a user I should be able to transform a standard Query Loop into a Curated Query Loop and maintain the same post template.

Currently, the content-search interface is provided by @10up/block-components. Update to use core SearchControl

• https://wordpress.github.io/gutenberg/?path=/docs/components-searchcontrol--docs

As an editor setting up a curated query loop, I'd expect this workflow:

1. Add a Curated Query Loop block to the post
2. Search For Post
3. Select Post
4. Search For Another Post
5. Select Post
6. ... and so on until the block is fully curated.

The current logic prevents this workflow because it is automatically updating the posts_per_page attribute whenever a post is selected, removing the search bar and preventing the user from easily searching for another post.

One solution is the always show the search bar, but this will prevent us from enforcing a limit.

Hi @psorensen

What are your thoughts on adding an option to autofill the posts if the curated posts are less than the posts.perPage or the default 10?
Let's say that an editor wants to show 5 posts, but only manually picked 2. With this option, the remaining 3 would be automatically fetched based on the query parameters?

Cypress is currently set up, but the tests are still boilerplate.

• Block variation can be added to a post/page
• Post can be added to the query loop
• Multiple posts can be reorganized with drag-and-drop

Steps to reproduce:

1. Create a standard query loop
2. Transform to a curated query loop.
3. Observe transform option doesn't have an icon.

As per what icon, I guess it should be the query loop icon?