Skip to content
Submitted 11 months ago

REST Countries API

Gerges Badr•310
@GergesBadr
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...REST Countries API with color theme switcher
  • HTML
  • CSS
  • JS
  • API
4advanced

Design comparison

SolutionDesign

Solution retrospective

Explore the world with my React app! in a user-friendly interface you will discover all needed information, easily.

Community feedback

  • snhase•200
    @snhasePosted 11 months ago

    Hello Gerges,

    I just finished this same challenge, your solution looks great and responsive!

    I have one suggestion, in your "Filter by region" feature. I noticed the button label doesn't update the region to the region selected onClick, remains "Filter by region" . How about passing the region item value that you get from the mapping over regions list, in the handleSelectedRegion function to set SelectedRegion value instead of the event? like so handleSelectedRegion(region)?

    Hope this helps! Happy Coding :) snhase

    0
    Gerges Badr•310
    @GergesBadrPosted 11 months ago

    Hello snhase, I really appreciate your time reviewing my project and your feedback.

    It sounds nice to me, good notice. I will consider edit this as soon as possible. Also, I looked at your solution and it also looks great. Nice work :)

    0
    snhase•200
    @snhasePosted 11 months ago

    @GergesBadr thanks!

    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord