Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

REST Countries built with vue3 and scss

Abiodun Olunuโ€ข 70

@abiodunolunu

Desktop design screenshot for the REST Countries API with color theme switcher coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
  • API
4advanced
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi Everyone, I would entertain feedbacks, constructive criticism and questions. Thank you.

Community feedback

@JesusAtao96

Posted

Hello Abiodun Olunu ๐Ÿ‘‹

Excellent work on this challenge ๐ŸŽ‰๐ŸŽ‰๐ŸŽ‰.

Try to integrate a debouce for the input search, this makes the request be called when the user stops typing. Currently in your solution for each letter written a request is being made and that is bad for a real project,.

More info

Your backend will appreciate it very much โœจ.

Happy coding ๐Ÿ˜Š.

2

Abiodun Olunuโ€ข 70

@abiodunolunu

Posted

@JesusAtao96 Thank you โ™ฅ

0
Jadson Santosโ€ข 235

@santos-jadson

Posted

Hello, Olunu! How u doing?

Your code is great and looks very similar to the original design. Congrats :D ... I think the only thing you could check is the fact that some of the cards don't have the same height. Maybe if you set a min-height or a max-height property, it could be fixed.

2

Abiodun Olunuโ€ข 70

@abiodunolunu

Posted

@santos-jadson Thank you, The main reason why I did it that way is because of the flags, they aren't the same heights and widths and I don't want to hide parts of any flag. But I will rectify this ASAP. Thanks

0
P
ApplePieGiraffeโ€ข 30,565

@ApplePieGiraffe

Posted

Hello there, Abiodun Olunu! ๐Ÿ‘‹

Nice work on this challenge! ๐Ÿ™Œ Your solution works quite nicely! I like the loading animation you added when waiting for countries to appear! ๐Ÿ‘

I just suggest adding cursor: pointer to the country cards perhaps giving them a maximum size so that they do not become too big in the desktop layout if only one country is shown. ๐Ÿ˜‰

Keep coding (and happy coding, too)! ๐Ÿ˜

0

Abiodun Olunuโ€ข 70

@abiodunolunu

Posted

@ApplePieGiraffe Thanks for the feedback, I will do that ASAP.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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