HTML5 and CSS3 , flex , hover

Community feedback

• 

  Nishkarsh Dubb•165

  @Nishkarsh01Posted over 3 years ago

  Hello, Gilcllys de Souza Costa! 👋

  Congrats on finishing another frontend mentor challenge! Here are few suggestions from me:

  • Update the background color of body to background: hsl(0, 0%, 95%);
  • Change the case of all the h1 headings to uppercase, using:

   h1{
    text-transform: uppercase;
  }
  

  • It is advised that you do not use inline styling for changing the size of SVG icons instead target them in the stylesheet, it is a bad practice as it makes code cluttery and harder to understand.
  • Try using utility classes to avoid repetition of code.
  • Try using comments more often in code, the comments stating the intent eventually help yourself and others clearly understand the logic behind using certain classes.

  Good luck with that, have fun coding! 💪

  0

  ---

  

  Gilcllys de Souza Costa•125

  @gilcllysPosted over 3 years ago

  Thank you very much, your comment will be very useful to my improve . I would like to thank for you reserve some minutes to look my code, this is very important to me.

  1