-
Notifications
You must be signed in to change notification settings - Fork 30
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
Adding Search button #104
Comments
Hi, @yitik-kawatra That's a great idea, but it will require ML no? Until obviously, we use the exact name for searching and loop through all the events present. Have you given the implementation any thought? Although, you don't absolutely have to, its a great idea! |
We will not require ML, It can be done by javascript and we can filter the events each time the person type a letter means on every letter pressed we can filter the events with the corresponding search. |
Sorry, that was a typo :( Its a great idea altogether, but I am afraid, implementation through javascript will really restrict it(?). Searching is something that many people adopt ML to do for greater efficiency and accuracy. Can you elaborate some more on your implementation? Do you mean, for eg. If I type |
@yitik-kawatra, the idea to implement search is great but I don't think we'll be able to implement it rn as the events name are all hardcoded and we don't actually parse it from a JSON file or any other data file to render it on the main page. The search feature would be doable if we create a JSON file that will have details of all the events and then loop through it to find the correct match. Doing it this way would also be a helpful step in taking the project toward Do Not Repeat Yourself principle by implementing this piece of code through a javascript function that loops through the data file and rendering the event card for every event instead of repeating that piece of code for every event. |
Like @soumyaa1804 explained, actually getting into it would entail a lot of changes in the current implementation. As for moving it towards a non-repetitive goal, it would be an enhancement that should be implemented regardless of this, and separately from this. I don't think it would be fair to have someone spend a lot of time making the relevant changes to implement this, when its not for the real website. It would be an amazing feature, and I am sure we could incorporate it at a later time. And if @yitik-kawatra would like to, we can hit him up when we do do it. But if you still would like to go ahead with this without changing a lot of the implementation, I think, it could be manageable if we just create an array or an object with hard-coded values in javascript. |
Yes i have tried to implement it through javascript but I also think that it will take lot more of time and i won't be able to give that much of time so will try another features which can be implemented in given time. |
That's alright too. |
I'm not sure if I should ask this here, please ignore if I shouldn't. I know how to filter search results by using JS but can anyone tell me how will we update the webpage after searching without reloading? (Not in this particular case, but in general) |
@tichnas, that's totally alright to ask it here. Here is a YouTube tutorial you can go through. |
Thanks for a quick reply. |
For #93
We can add a search button on the nav bar to search for the events by the event name.
The text was updated successfully, but these errors were encountered: