HTML and CSS

Community feedback

• 

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 3 years ago

  hey to get rid of the accessibility, change div class="footer" to footer class="footer", and wrap everything between footer and header in <main> :)

  0

  ---

  

  Victor•510

  @therealmaduanusiPosted almost 3 years ago

  @anoshaahmed Thanks Just fixed the errors 😁 How are you doing?

  1

  

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 3 years ago

  @therealmaduanusi yay. im doing good thanks for asking victor c:

  1
• 

  Tomi•565

  @Tomi-pterPosted almost 3 years ago

  Looks okay, but some few corrections I'd suggest:

  1. for the hover states of the social icons, I suggest you use the filter function to change the actual color of the icons. Check out this article on svg filter for more help
  2. In the mobile view, you should add some inline padding to the sections to give them some space from the edges.
  3. Check out your solution report for accessibility and html issues.

  Hope this helps 👍🏾

  0

  ---

  

  Victor•510

  @therealmaduanusiPosted almost 3 years ago

  @Tomi-pter thanks honestly have being thinking of how I can fix the hover color. Also the accessibility keeps talking about the landmark 🤷

  1