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
Better sorting algorithm while searching parent pages #29143
Conversation
Size Change: +681 B (0%) Total Size: 1.38 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and looks good. It feels a bit weird that getItemPriority
returned bigger numbers for items we want lower position, but overall everything seemed fine :)
@youknowriad didn't get a chance to test this; overall seemed fine. Thanks for the improvement. Thinking we should add a similar toggle to author selector here so that if the site has a small number of pages, we use a traditional HTML selector vs the searchable selector. |
@adamsilverstein make sense, we might even consider making that behavior built-in with a boolean/threshold prop in ComboboxControl maybe |
This PR improves the search algorithm for parent pages by adding some client logic to: