Skip to content
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 sorting functionality back to the gitcoin explorer #570

Closed
owocki opened this issue Mar 8, 2018 · 9 comments
Closed

add sorting functionality back to the gitcoin explorer #570

owocki opened this issue Mar 8, 2018 · 9 comments
Labels
bug This is something that isn't working as intended. frontend This needs frontend expertise.

Comments

@owocki
Copy link
Contributor

owocki commented Mar 8, 2018

#523

this was functionality that existed in the old design. you could sort by difficulty, creation time, and value

@owocki
Copy link
Contributor Author

owocki commented Mar 8, 2018

@thelostone-mc would you be up to take this? happy to tip you

@PixelantDesign
Copy link
Contributor

@thelostone-mc for reference:
screen shot 2018-03-08 at 12 32 25 am

@thelostone-mc
Copy link
Member

thelostone-mc commented Mar 8, 2018

Consider it done ^_^
Will get this in later tonight

@PixelantDesign PixelantDesign added in progress bug This is something that isn't working as intended. labels Mar 8, 2018
@thelostone-mc
Copy link
Member

@PixelantDesign Do we wanna add Tech Stack too ? cause this list could be super long as more and more bounties come in

@owocki
Copy link
Contributor Author

owocki commented Mar 8, 2018

we have a list of tech stacks in the Keyword model... to adityas point, i dont know if checkboxes are the right input for that though..

@thelostone-mc
Copy link
Member

Also wouldn't sort boxes be radio buttons?
And would the sort function be in descending / ascending ? Ideally it should provide both
cause it's a sort filter

@PixelantDesign @owocki

@PixelantDesign
Copy link
Contributor

Yeah, not sure why I put that there, since checks suggest ability to select multiple. i’ll move it back where it was.

The tech stack however I intended to have on the left side connecting it with the search. This was the main driver behind this task since it effects the content that changes on the page.

@thelostone-mc
Copy link
Member

thelostone-mc commented Mar 8, 2018

Moving it back. Thoughts?

help

cc @eswarasai @KennethAshley

@mbeacom mbeacom added the frontend This needs frontend expertise. label Mar 8, 2018
@eswarasai
Copy link
Contributor

@mbeacom @PixelantDesign -- I guess this can be closed with #790

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is something that isn't working as intended. frontend This needs frontend expertise.
Projects
None yet
Development

No branches or pull requests

5 participants