HTML, CSS

Community feedback

• 

  Zuzki•180

  @LuisDGraciaPosted almost 4 years ago

  Hi, I see you made the design but there are some things you could do to improve, having using a position relative/absolute on the body is not the ideal solution since the more elements you add more work you'll need to do on the positioning, I suggest to you to learn about CSS flexbox and CSS grid that could improve a lot your CSS and reduce the lines of code you use.

  I see you have "font-family: 400", font-family stays for the name of the font, like Arial, sans-serif and so, font-weight only accepts 1 value you can't use 400, 700.

  using a table for the footer is not a bad idea, but not the ideal, you could encapsulate the 3 elements each one on their div container, and use a display flex yo put then on the center.

  I hope this could help you to improve and remember to check the report to get more info about you could fix

  1

  ---

  

  Tejveer Singh•10

  @TejveerSinghRajputPosted almost 4 years ago

  @LuisDGracia Hi, Thanks for giving feedback and, I am new in front end developer. I will improve my solution.

  0