Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Panel] Pages field: Query including subpages #217

Closed
lukasbestle opened this issue Mar 12, 2019 · 1 comment

Comments

@lukasbestle
Copy link

@lukasbestle lukasbestle commented Mar 12, 2019

Current behavior

Let's say a custom query is used for a pages field like so:

related:
  title: Related pages
  type: pages
  query: site.homePage.children

The pages field will now display a list of the pages in the queried collection. Unfortunately, the full page tree is no longer displayed (because of this line), so it's no longer possible to select subpages. Only the pages that are directly in the queried collection can be selected.

Feature idea

This may be intended for some use-cases, but in others it might make sense to select subpages as well, e.g. if you want users to select a page from a custom collection that is a filtered tree structure of pages.

Possible implementations

I think this should be a blueprint option of the pages field.

  1. Option 1: Additional blueprint option
    related:
      title: Related pages
      type: pages
      query: site.homePage.children
      querySubpages: true
  2. Option 2: Nested option
    related:
      title: Related pages
      type: pages
      query:
        query: site.homePage.children
        subpages: true
@bastianallgeier bastianallgeier added this to Interesting in Roadmap Apr 16, 2019
@afbora afbora referenced this issue Jul 1, 2019
2 of 4 tasks complete
afbora added a commit to getkirby/kirby that referenced this issue Oct 3, 2019
@afbora afbora referenced this issue Oct 3, 2019
4 of 4 tasks complete
bastianallgeier added a commit to getkirby/kirby that referenced this issue Oct 10, 2019
bastianallgeier added a commit to getkirby/kirby that referenced this issue Oct 10, 2019
@bastianallgeier

This comment has been minimized.

Copy link
Contributor

@bastianallgeier bastianallgeier commented Oct 10, 2019

Roadmap automation moved this from Interesting to Done Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
  
Done
2 participants
You can’t perform that action at this time.