-
Notifications
You must be signed in to change notification settings - Fork 40
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
Remove Search Button #58
Comments
close the issue |
Hi @ayushpattnaik , This was discussed here. Thanks! |
hi @monddelmundo |
Hi @ayushpattnaik , |
Okay great!! |
Is this still open? |
Yes @BlckShadow . Waiting for PR #59 to be reviewed and merged. |
Ok I was just wondering since I'm a beginner and I want to take a react app since I'm new to react and I want to challenge my for final PR |
@monddelmundo Another q: Is only the button to remove and not the search field Correct? Would that make the search field useless? |
Hi @BlckShadow , No, the search field is functional even without the button. You can try it here: You can look for the other issues posted for this project and try to solve those, like beautifying the site (#5 ). |
@monddelmundo Ah ok that's awesome |
Ok I would love to take this on for my final PR for Digital Ocean Hacktober for my final PR |
Ok, I also submit one for PR for this issue; thanks again for the attempt and learning experience with this hack fest |
@ayushpattnaik Hi I submitted a puller request for this issue but it still pending so I'm not sure what to do |
I guess the repo is inactive |
Hi, I will be removing the Search button.
The text was updated successfully, but these errors were encountered: