-
Notifications
You must be signed in to change notification settings - Fork 113
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
Add sidebar with GUI filters to the search results page #103
base: master
Are you sure you want to change the base?
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@googlebot I signed it! |
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
nice! I usually do the reviews from Gerrit though - see CONTRIBUTING. Would you mind sending me a change there instead? |
dc647d5
to
04c3621
Compare
There's an error adding my GitHub email address so I'm unable to push the changes to Gerrit. |
I had a quick look at this last week. I have two complaints:
I think this feature would make more sense as a dropdown menu. |
@hanwen noted. I'll put in some changes soon. |
ping? |
Hi, I got busy with work and completely forgot about this. I'll work on this tonight. Thanks for the reminder. |
sorry for the delay, i have been busy with other things (sending through Gerrit as described in CONTRIBUTING usually yields faster feedback).
|
implementation issue: you look at substrings ("lang:XXX") in the backend to populate the checkboxes. This is incorrect if those bits appear as substrings eg. clang:c In the server, you can have access to the parsed query, so you should instead find the toplevel query.And and then look at its children to find all query.Repo/query.Language atoms. |
it also needs some work so the repo list view doesn't get messed up. The drop down shouldn't be there for the repolist. --- FAIL: TestBasic (0.00s) |
This PR adds interactive filters in the sidebar to filter between repositories and languages.