Skip to content
OverviewSolutions
3
Comments
2
JustynaKorzycka
@JustynaKorzycka

All comments

  • Robin Bosch•150
    @robin-boschSubmitted 10 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
    JustynaKorzycka•140
    @JustynaKorzyckaPosted 10 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
  • JustynaKorzycka•140
    @JustynaKorzyckaPosted 10 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