React solution for REST Countries Challenge

Community feedback

• 

  Fury•945

  @ashiqfuryPosted almost 3 years ago

  Fantastic work..👌

  • When I first type India it does not show up, I thought there is some bugs on your code.
  • I reviewed your code, I noticed you compared the country name and region. It really a good thing.
  • Then I changed region to Asia and tried again, worked.
  • I really learned so much things on your code.

  I write a simple and clean code.. I liked that.

  • Reduce the transform effect, subtle scaling animation will look pleasent.
  • Use lazy loading, to improve performance, because there some many images to load at a time.

  I hope my suggestions will help to improve your future projects.

  Thank you.

  Marked as helpful

  1

  ---

  

  Jeremy Auvray•190

  @jeremy93-2008Posted almost 3 years ago

  @ashiqfury Thank for you feedback, I try to replicate you bug but I was not able to do it.

  The transform was to make some animations but I try it on a Macbook M1, so maybe the experience may not be fluid for all that for sure, thanks for the advice.

  For the lazy loading, I think about implementing it, but laziness kill me and as the webpage doesn't show any performance issue I decide to no implement it, but for sure if it will be a professional project I would need to implement it

  Thanks :)

  1