Benjo Quilario• 1,810
@benjoquilario
Posted
Hey @ahemadShaikh. Great job on finishing this one. It looks good and response rather well.
Some suggestion:
- All page content should be contained by landmarks. It will better if the structure of your html page is like this
<header> </header>
<main></main>
<footer></header>
- Don't forget to add a alternative text on your
<img>
tag when user having a internet problem. the logo must have a alt="manage logo" something like that, and also add the alt on the person image with there name. - Every
section
need aheadings
h1-h6 - The footer link also should have anchor element <a>. Because this element is also going to transfer user somewhere. <a href="#">
<img> tag here
</a> - You must add the
e.preventDefault()
on your form to prevent the page from refreshing
Good luck and keep coding
Marked as helpful
1
Ahemad Shaikh• 80
@ahemadShaikh
Posted
@benjoquilario Thank you for your suggestion, i'll definetly follow this suggestion in projects.
0