-
Notifications
You must be signed in to change notification settings - Fork 115
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.