HTML, CSS Grid, flex and media queries

Community feedback

• 

  Lucas 👾•104,160

  @correlucas

  Posted almost 3 years ago

  👾Hello Adriel , Congratulations on completing this challenge!

  Your solution is just perfect! I liked a lots what you did here using the picture to wrap the images and making the container full responsive. Really good solution! Something that can be a time saver for you is to use a CSS RESET to remove all default settings for margins, making the images easier to work, see the article below where you can copy and paste this css code cheatsheet: https://piccalil.li/blog/a-modern-css-reset/

  I saw that for some properties you’ve used rem and for others px. In this case it is better to use only one kind of unit to have a better organization for your code. relative units as rem or em that have a better fit if you want your site more accessible between different screen sizes and devices. REM and EM does not just apply to font size, but to all sizes as well.

  ✌️ I hope this helps you and happy coding!

  10

  Marked as helpful
• 

  Laharl•1,000

  @UrbanskiDev

  Posted almost 3 years ago

  Hello Adriel Santos Costa !

  Congratulation for finishing the project

  Some suggestions to make it better :

  1. You can add a little hover effect to your button to make it less rigid !

  .card-button:hover{
  background-color: hsl(158, 36%, 32%);
  }
  

  You can also add a transition to make the hover smoother, I give you a link about transition if you want to learn more about the topic :

  • MDN Transition

  You did a good job !

  I hope it helps you, keep learning and happy coding !

  10

  Marked as helpful