Adann Jacinto• 790
@DanCodeCraft
Posted
Well done, @Depaulaeduardo!
You inverted the order of the cards in the middle. To avoid this, it's generally advised to work on the mobile version first, so the order is always the right one.
I can also point out that you could consider changing your media query. It creates a weird layout in between screen sizes (the cards are not flexing, overlapping the screen, and the text in the middle becomes a huge pillar of letters. If I was seeing this on a tablet, I wouldn't be able to understand the content.
Some points you should consider on your CSS:
- A modern reset, so it fits every browser out there.
- Researching different CSS measure units, to avoid issues on bigger projects - the font size is a good example: use rem, never pixel.
- Try to use more descriptive names in your classes. If you open this same project 2 years from now, would you be able to understand it right away?
Keep up the good work!
Marked as helpful
0
