-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Nav Block: URL item in the search results should appear first #18425
Comments
My concern with showing it at the top is that it would/could push down more relevant content (post/pages) results. Random question that this brings to mind: Is there keyboard navigation integrated? That is, can I use the up/down arrows on my keyboard to navigate the results? Another related question: The "Press ENTER" message should likely only display when its the only results — otherwise, I'd expect to be able to use the keyboard to navigate results and press enter to insert any item. |
What if instead of showing the link option in the list, it waits for the first dot in the URL before showing? It seems strange to me to show UI for adding a link when I've typed "hello", that's not even a valid URL yet. |
@shaunandrews Thanks for these observations.
Yes, you use the arrows to move into the search results.
You can do exactly what you describe so it is valid. That said, as we expand the possibilities for the data used to back to search suggestions the
@apeatling I think the wider issue here is refining the valid "URL" detection. Remember however that direct entry encompasses
...and it might be that the "dot" detection wouldn't work in that scenario. Did we decide if we're happy with the position of the URL or we want to move it? |
Happy to chat on a different issue about this one. I think only triggering on exact matches like |
I think in its current state it has to show at the top, even if it pushes relevant content down further. That is still much better than the mystery of pushing the enter key when it's not visible in the popup (because it's at the bottom of the list). |
Always showing the URL result at the top doesn't feel right. Look at this scenario, where I've searched for the word "Off": The "Offers" page is the result I'm looking for, but the URL result of "Off" is presented first. In this case — and with most input that doesn't look like a valid URL (i.e. searching) — presenting the URL result is unhelpful. -- That said, I think we're looking to address a few different problems here:
To solve #1, I think we could try parsing the input to attempt to determine if its a valid(-ish) URL. If so, then show the URL result first. To solve #2, I think we should change the text to something simpler: "Add a link to this URL". To solve #3, change the text as mentioned above and ensuring that the first item in the list is defaulted to active — that is, hitting Here's those three things visually: |
As we seem to have design feedback and I agree with @shaunandrews, going to remove that label for now. |
Prioritizing local sources seems correct to me. |
Created from #18135.
This is not a definitively confirmed task (ie: use of "I wonder..." in issue).
It would be good to get a view from @karmatosed and @shaunandrews before we proceed here.
The text was updated successfully, but these errors were encountered: