Country Rest API with React, NextJS, Framer Motion, MaterialUI...

Community feedback

• 

  Clarence Onumajulu•400

  @clarencejuluPosted over 2 years ago

  Hi Nathan, Firstly, great work on this project. I really like the 'up-down' animation you added on the first load.

  However, I did identify two minor issues:

  1. Border Countries - Your 'border countries buttons' navigate perfectly, but if you want to show the country's full name on each button, you can do the following,

     for(let index = 0; index < countries.length; index++){ if(countries[index].alpha3Code === border){ country = countries[index]; break; } }

  Here 'countries' is an array of all the countries generated from the API, while border was passed in as a prop into the current component this loop is in.

  2. Responsiveness - The responsiveness of your project is great in general but the header( the part which contains 'where in the world' and 'dark mode' ) could do with a little work. I think you should reduce the font-size value when the screen width is reduced so it looks good on smaller screens. Then when the width of the window is at 599px and below, the countries look stretchy. I think you should allow two countries to be on the same line here.

  GoodLuck!

  Marked as helpful

  1
• 

  Nathan Lowe•450

  @NathLowePosted over 2 years ago

  Thank you very much for this very helpful comment of work.

  1