-
Notifications
You must be signed in to change notification settings - Fork 203
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
Search URI is not pushed to the address bar #399
Comments
👍 |
I wonder if we could send a flash message like: "(Log in) to see search results. Or see the (search on GitHub)". Stuff in parens would be hyperlinks. |
In the case of non-authentificated user we should send:
|
Sounds like a good plan. |
Also need to include params like |
Making progress on this. Pushing params to the URL is all done--and toggling filter params as well. I just need to hook up reading from the URL in And then we need to tackle #516 (actually having and reacting to a state for the pushState). I don't think it will make much sense to deploy one without the other. Or at least it would feel very broken. |
heh, Karl was already on it :) |
OK, I think I've got this one done. But I need to write some tests before sending a PR--there's a billion things to keep track of, which means I'll surely break it in the future. ^_^ |
Step to reproduce:
Expected:
to have the URI in the address bar to be something along (Not necessary this parameter syntax, just an example)
https://webcompat.com/issues/search?is=issue&is=open&q=google&in=title
Current:
The address is
https://webcompat.com/issues
Why it's important.
It's currently not possible for someone to share a specific search URI on the Web or by mail.
Possible issues
When someone is not logged in to webcompat.com, the search URI will not be working. We could handled that case through a redirect to github search.
The text was updated successfully, but these errors were encountered: