Responsive site

Community feedback

• 

  Grego•1,310

  @Grego14Posted 7 months ago

  Hello! 🎉 congratulations on completing the challenge! 🎉

  I think you meant: margin: auto 0; and not margin: auto o; in the body.

  You can skip the third value of the box-shadow since you don't need it!

  box-shadow: 10px 10px black;
  /* or */
  box-shadow: 10px 10px;
  

  Try using the alt attribute to specify what semantic meaning the image has, if the image is decoration it is better to leave the attribute empty.

  Do not skip the headings, you are using first an h2 and then an h1, and I also recommend that you only use them for titles or subtitles, you can use a div or a p for the other texts.

  I recommend that you increase the size of the fonts since it becomes very difficult to read the text, and also use another font as an fallback if the main one does not load.

  I hope this helps! 😁

  0