Skip to content
Submitted almost 2 years ago

Rest Api Countries App - React, SCSS, Framer-Motion, Axios

Joanna Skrzypczak•530
@joaskr
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

#The challenge is still in progress and will be improved

I added the solution to FE mentor to check the accessibility report. It's definitely not done yet and will be improved.

There are still some errors in responsiveness and I plan to do a lot of code refactor because it's very messy. SCSS code also needs some fixing - I started with desktop-first approach and in the middle of development switched to mobile-first approach and that resulted in code duplicates and in big mess.

So to sum it up - I know that it's broken but I'll fix it some day :)

Community feedback

  • Karthik suryadevara•825
    @karthik2265Posted almost 2 years ago

    Hey joana,

    The solution looks good and the responsiveness also works well.

    few good things I noticed in the code.

    • The components and the folder structure looks organized, well done.
    • Framer motion works very smoothly, good choice
    • Good use of context for theme
    • Since the data is filtered on the backend it's a great idea to use debounce.
    Marked as helpful
    0
    Joanna Skrzypczak•530
    @joaskrPosted almost 2 years ago

    @karthik2265 Thanks for the comment 😊

    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