Responsive landing page using Semantic HTML and flexbox

Community feedback

• 

  hitmorecode•6,680

  @hitmorecodePosted about 1 year ago

  Nice well done and you did mobile first approach very good. I have a few tips

  • Make it a habit of using min-height: 100vh; on the body. This is for the responsiveness of the page.
  • For mobile use background-size: contain;
  • Don't add padding or margin on the body
  • You should place everything inside a container and center the container on the body.
  • The social media icons are not round. Use a <div give make the width and height equal and place the icons inside the div.

  See if you can manage it, if you need any help let me know

  Marked as helpful

  1

  ---

  

  Ahmed Abd Allah•180

  @Ahmed7019Posted about 1 year ago

  @hitmorecode i really appreciate it and thank you for your precious time . Specially for the social media icons tips as i struggled while trying to solve it .

  0