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 API Using SvelteKit

#svelte
Ivan 2,630

@isprutfromua

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


please tell me how to make my code better. Thanks a lot

Community feedback

@besttlookk

Posted

hi, Following are the issues I like to point out:

  1. Cards are not well organized. I would suggest using grid .
<div class="container">

</div>
.container{
  display:grid;
  grid-template-columns: repeat(auto-fit, minmax(280px,1fr))
}

With above code you dont have to write media-query for differnent sceen just to style cards.

  1. fix the header while scrolling.
header{
position:sticky;
top:0;
}
  1. On Detail page border country name is almost toching the bottom of the screen. add some padding.

Good luck, #happyCoding

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