@AntoineC-dev
Posted
Hello Clarence, what's up ? Good job completing this challenge 🎉🎉🎉 All basic styles (colors, typography, etc...) seems to be correct so nice work!
I have 3 advices for you:
- You should add the property
cursor: pointer
to all interactive elements, in this case all the links and buttons. This is a small detail that goes a long way for better user experience.
Now to fix the issues in your report:
-
You should wrap both sections (home & about) in a
main
tag. You did it already for the header which is great! -
Same for the
<div id="wrongValue" class="notification"></div>
. The parent container should be anaside
element. It is the landmark you should use for all the elements outside ofheader
,main
orfooter
.
Have a nice day/night. Peace 😊
Marked as helpful
@clarencejulu
Posted
@AntoineC-dev You are right bro...I need to add these. It is essential to make sure html is sematic. Thank you so much bro. It's always great to hear from you, do have a nice day/night!
@clarencejulu
Posted
@AntoineC-dev I just fixed the issues that you outlined in my HTML. Thanks as always. I really appreciate your input bro...