baa76 / bwa-u4-feature-request

0 stars 0 forks source link

Feature Request Summary #2

Open thecodeduchess opened 6 years ago

thecodeduchess commented 6 years ago

Grade: Meets Expectations

Summary: I enjoyed looking at your feature request and thought it made a lot of sense. The current implementation for codecademy's sample allows users to search without being logged in. When a visitor accesses that token page, they might think the app is broken or some bug is occurring. Therefore, the app should also disable the ability to search if not logged in. The search button could be greyed out.

You lay out a rock solid plan for implementing a feature that definitely improves the user’s Jammming experience. The idea itself is a natural suggestion, as most users would likely assume the Enter key would serve that purpose and therefore would find it inconvenient that the app doesn’t do that. Your implementation is also the easiest way to accomplish the goal.