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

Navigation Link: Prioritize pages in the Link Control suggestions #53904

Closed
scruffian opened this issue Aug 24, 2023 · 1 comment · Fixed by #54622
Closed

Navigation Link: Prioritize pages in the Link Control suggestions #53904

scruffian opened this issue Aug 24, 2023 · 1 comment · Fixed by #54622
Labels
[Block] Navigation Link Affects the Navigation Link Block [Type] Enhancement A suggestion for improvement.

Comments

@scruffian
Copy link
Contributor

scruffian commented Aug 24, 2023

What problem does this address?

When I insert a new Navigation Link block I am most likely trying to add a page to my navigation. At the moment the list of suggestions in Link Control is sorted by most recently edited, I believe:
Screenshot 2023-08-24 at 10 19 55

What is your proposed solution?

Instead we should prioritize pages in this list, and only show other content if no pages match the query. I tried this in #53903, but ran into some problems.

@scruffian scruffian added [Type] Feature New feature to highlight in changelogs. [Block] Navigation Link Affects the Navigation Link Block labels Aug 24, 2023
@draganescu draganescu added [Type] Enhancement A suggestion for improvement. and removed [Type] Feature New feature to highlight in changelogs. labels Aug 24, 2023
@draganescu
Copy link
Contributor

Informed by #50432

  • show only pages in the default suggestion list
    • ideally try a scrollable tree of pages reflecting the page hierarchy - but this is not clear yet (e.g. what if there are hundreds of pages).
    • we can start with just listing pages ordered by last modified desc
  • when a user searches search in everything that is supported, but prioritise pages if the match is in the title

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Link Affects the Navigation Link Block [Type] Enhancement A suggestion for improvement.
Projects
Development

Successfully merging a pull request may close this issue.

2 participants