REST Countries API with color theme switcher - Vanilla JavaScript

Community feedback

• 

  Praneet Dixit•120

  @PraneetDixit

  Posted about 4 years ago

  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!

  01

  Marked as helpful
• 

  

  Chamu•13,860

  @ChamuMutezva

  Posted about 4 years ago

  Well done.

  • When I make a continent selection, there is no option to return to all countries
  • I would prefer the bordering countries to be full names - the short code might not be helpful to users

  Happy coding

  02

  Marked as helpful
• 

  P

  Patrick•14,265

  @palgramming

  Posted about 4 years ago

  ⭐⭐⭐⭐⭐ Wonderful Job

  01