Submitted over 1 year ago
Dictionary Web App | Vite React TypeScript TailwindCSS
@ShivangamSoni
Design comparison
SolutionDesign
Solution retrospective
All Feedbacks are Welcome 🔥
Community feedback
- @towkirPosted over 1 year ago
Loved the idea of content placeholder while api request is going on, great work. I have some feedbacks though:
- I'd recommend not clearing the search input after pressing enter or clicking the search icon, it helps when you're showing the "No definition found" message, keeps a track of the searched word.
- I searched for
asdfa
and observed the network panel, there are at least 4 api calls when words are not found, (404 responses). - And then toggled to mobile view, typed nothing, just focused the search input and there were another 4 calls again.
- Changed back to desktop view and the same happened.
- The github repo is not visible, probably made private or something? you can check by opening the repo link from an incognito window.
Cheers !!
Marked as helpful1@ShivangamSoniPosted over 1 year ago@towkir thanks for the feedback.
I'll fix those issue ASAP.
Update: I Fixed the above listed issues.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord