Skip to content
Submitted over 3 years ago

Responsive Country Finder with Vanilla Javascript and REST Country API

nweke joseph•260
@nuel1
A solution to the REST Countries API with color theme switcher challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • Carl Wicker•1,055
    @carlwicker

    Hey ya, just been checking out your solution. There are a few issues, the most important is the speed. There seems to be bottleneck somewhere, as the data is very slow to return.

    It might be worth getting the all API data once and storing it (since it doesn't change) to reduce APIs calls. Maybe that slowing it down, but haven't looked into your code.

    Also...

    • search functionality is not working from the input field.
    • consider adding a option to clear the filter on the dropdown, i know its not in the design but it makes sense to be able to clear the filter, I believe this was overlooked by the designer.
    • finally, fix those accessibility and HTML issues in the report.

    Layout is looking good, I like your tweaks to the original design.

    Keep up the great work.

    Marked as helpful

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