Aleksei• 200
@AlekseiBestuzhev
Posted
Hello.
- I would remove the styles from the root element as its only function is to accept components. The header should be 100% viewport and not have padding from its edges.
- I would add one more media query for the cards, since everything is very compressed at a resolution of more than 767 pixels. Change the number of grid columns.
- On the details page, if there are many countries, then they go beyond the viewport and create a horizontal scroll. Use
flex-wrap: wrap;
- And in the same place you use <p> in which buttons lie, although it is intended for text. Use <div>. This will be more seductive.
- Missing region Antarctica.
- Not all properties exist in all countries. The capital of Antarctica, for example. You need to handle their absence.
Well done for completing this challenging project. Good luck!
Marked as helpful
0