Response landing page with flexbox and boostrap grid

Community feedback

• 

  Suraj Raju Hadage•670

  @SurajHadagePosted over 2 years ago

  See my HTML file to shorten your HTML file and make it Semantic and accessible. I have some suggestions for your footer, for footer logo use <figure> tag so your you do not have to use display: block; To align icons of first column of your footer use fontawesome (https://fontawesome.com/docs/web/style/lists). you have 4 <ul> inside .footer-links. To shorten your CSS use .footer-links > display: flex; --> let say you have 4 <ul> with classes .col-1, .col-2, .col-3, .col-4 respectively. .col-1 { flex-basis: 30%;} .col-2, .col-3, .col-4 { flex-grow: 1;}. Make sure to give circle border to social links with .col-4 > li > {border-radius: 50%; height: 30px; width: 30px}. Your icon are inside the <a> tag, remove it, Its not good.

  Marked as helpful

  0