All comments

• 

  P

  Robin Bosch• 150

  @robin-bosch

  Submitted 8 months ago

  I did this one a bit differently, not using the API and using the JSON file with the countries directly, due to performance issues with the API. I wrote a small Next.js backend replacing the API. First time using the app router of Next.js, let me know if I can improve on the usage of the app router, especially with the usage of the "use client" declaration.

  

  Where in the world - Using NextJS in TypeScript and the new app router

  1

  80

  

  JustynaKorzycka• 140

  @JustynaKorzycka

  Posted 8 months ago

  Nice work :) I found a small bug in your application: if I choose for example Curaçao there will be error. I'm guessing the problem may be that you're passing a foreign character to the url.

  1

  This comment has received 1 replyShow
• 

  wafash08• 130

  @wafash08

  Submitted 8 months ago

  Your feedback is highly appreciated 🙌

  

  Responsive Manage Landing Page

  2

  40

  

  JustynaKorzycka• 140

  @JustynaKorzycka

  Posted 8 months ago

  Hi :) I found few things that could be improved:

  1. there are no horizontals paddings/margins - for some widths content is on the edge of the page.
  2. For mobile devices You can add closing the menu when clicking outside the menu area
  3. I suggest shortening the homepage code: exports static data to another file, create new components: heroSection, preFooter e.tc.
  4. pay attention to the heading hierarchy :)

  Marked as helpful

  1

  This comment has received 2 repliesShow