@PraneetDixit
Posted
Hi @Tobesh01 !
Good Job on the challenge !
Some points for improvement -
- There is no way to know which region have you selected once you click and select any region. It would be great if the name of the selected region is indicated in the place of "Filter by Region".
- I noticed that you are sending a separate request every time user is changing the filters. A better approach can be retrieving all the data only once on the initial page load, store it in a variable and do all the filtering operations on the stored data.
- The name filter and the region filter are not coordinating. When I type "ind" (without quotes) in the filter box and select the region Asia, I expect the app to show me all the countries in Asia whose name contain "ind". But both filters are overriding each other.
- Since there are many countries and each country card has an image, loading all those images can leave a bad effect on the UX. You can set
loading="lazy"
on the images so that the off-screen images are not loaded until the user scrolls to them. You can read about lazy-loading here.
Happy coding!
Marked as helpful
@blade-01
Posted
@PraneetDixit Thanks for the feedback.
Your points are very well noted