Pull-Request-Community / pull-request-community-website

Pull request community
https://pullrequest.co.il/
MIT License
56 stars 100 forks source link

Added filter capabilities #69

Closed asharonbaltazar closed 3 years ago

asharonbaltazar commented 3 years ago

(Due to @MichalPorag's instructions, I left out the search functionality). Also due to inactivity, I deleted this repo last week and had to recreate the components from scratch 😬, but I didn't want to lose this chance for a PR. Lesson learned.

This PR adds a dynamic dropdown component that sifts through people. To avoid adding too many conditionals in the people.map, I took an abstract route and decided an array with all the filters is the way to go. This way, any future contributor can simply add a similar set of data and all filtering capabilities for that new dataset will be automatically enabled, as well as a new dropdown component representing that data (with all included functionality).

binarica commented 3 years ago

@asharonbaltazar could you squash the commits into one?

asharonbaltazar commented 3 years ago

כן, אין בעיה, אבל מאוחר יותר.

@binarica Done 👍

ozizhakGT commented 3 years ago

hi @asharonbaltazar please add responsive design tnx alot!