PA Review: Band/Album Search

[omarnyte]

• [x] Backend: DB, model, controller, views
• [x] Redux Loop: ajax, actions, reducer
• [x] Presentational Components
• [x] Styling
• [x] Smooth, bug-free navigation
• [x] Adequate and appropriate seeds

[omarnyte]

Thank you for the suggestion to wrap every letter of my search query with '%'. It really improves the search functionality!

[niartenyaw]

Awesome!!

Functionality

Perfect! Two suggestions:

• [x] You are currently waiting for the user to push enter before starting the search. after that, you have the dynamic look up by going to /search. You could immediately go to /search when they enter the text box. take a look at the onfocus callback of inputs.
• [x] Right now, you are sending an ajax request on every key press. If I type 4 letters super fast, that is sending 4 requests, but I really only care about the last one. There are some clever things you can do with setTimeout that allows you to submit a request only if there have been no changes in the last, say, 400 ms. Let me know if you want an explanation of how!

Styling

• [ ] I feel like everything could be spread out horizontally. There is a lot of unused room. Maybe have all info one a single line and space it out a bit more. Kind of like how Spotify displays songs?
• [x] Might want to have some kind of separator (even just a thin line) between search results to better separate them.
• [x] It might be nice to be able to click the entire SearchItem to go to the correct page, not just the name. Something to consider.