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

HTML5 and CSS3 , flex , hover

@gilcllys

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I would like someone to check if the site correctly responsive.

Community feedback

@Nishkarsh01

Posted

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

Posted

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

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