Responsive landing page with HTML and CSS flexbox

Community feedback

• 

  Davisnz•150

  @DavisnzPosted 8 months ago

  Getting there!

  Feedback:

  1. Don't forget to center the box in the screen, you can do this by using flexbox attributes and adding padding.

  2. For the links, you added a div class called "links". Because they will be links, you should use anchors for your links, not divs, so instead of

  <div class="links">GitHub</div> you should use:

  <div id="links">
  <a href="#" class="social-links" >GitHub </a>
  <a href="#" class="social-links" >Frontend Mentor </a>
  <a href="#" class="social-links" >LinkedIn </a>
  <a href="#" class="social-links" >Twitter </a>
  <a href="#" class="social-links" >Instagram </a>
  </div> 
  

  And take it from there :)

  Marked as helpful

  0

  ---

  

  Carlos Arroyo Becerra•30

  @CarlosArroyoBPosted 8 months ago

  Thanks for the help! I will take it into account for future projects. :)

  1

  

  Davisnz•150

  @DavisnzPosted 8 months ago

  @CarlosArroyoB

  no problem! I would highly recommend you to fix this one so you can nail the process :)

  Marked as helpful

  0