Responsive page - 4cards mobile/desktop

Community feedback

• 

  P

  João Víctor de Araujo Lima•350

  @xuaunPosted about 1 month ago

  Hello Mateus,

  Congratulations on your work on this project! I believe you did very well. I hope my comments help you evolve in your future projects.

  Main difference

  The main point that caught my attention was the lack of padding or padding-top in your body.

  Code semantics

  I saw that you included all the texts in your code as h1-h5, I believe that this is not the best choice for the semantics of your code. One suggestion would be to leave your entire title as h1 (and not part h1 and part h2) and the titles of the cards as h2 and everything else as p. In the main title you could have used another semantic tag to differentiate the texts, such as <span>, <b> or <strong>.

  Other comments

  Other things I noticed were:

  • Shadow color slightly different from original design
  • Main title smaller than in original design
  • Some card size smaller than in original design
  • Spacing/margin between <header> text and cards smaller than in original design
  • Top collored border of cards larger and more curved than in original design.

  Keep it up and you will go far in your future projects!

  0