@0xabdulkhaliq
Posted
Hello there š. Congratulations on successfully completing the challenge! š
- I have other recommendations regarding your code that I believe will be of great interest to you.
HEADINGS ā ļø:
- This solution consists incorrect usage of
<h3>
so it can cause severe accessibility errors due to incorrect usage of level-three headings<h3>
- Every site must want only one
h1
element identifying and describing the main content of the page.
- An
h1
heading provides an important navigation point for users of assistive technologies, allowing them to easily find the main content of the page.
- In this solution there's
<h3>
element which is this<h3>Improve your...</h3>
, you can preferably use<h1>
instead of<h3>
. Remember<h1>
provides an important navigation point for users of assistive technologies so we want to use it wisely
- So we want to add a level-one heading to improve accessibility
- Example:
<h1>Improve your front-end skills by building projects</h1>
- If you have any questions or need further clarification, and feel free to reach out to me.
- If you have any questions or need further clarification, you can always check out
my submission
and/or feel free to reach out to me.
.
I hope you find this helpful š Above all, the solution you submitted is great !
Happy coding!
Marked as helpful
@newtothis90
Posted
@0xAbdulKhalid I used the <h3> for sizing purposes only. What if I was to remove the <h*> element completely and just wrap the text in a different element, such as <p>, and then adjust the size with that? Would that be a better way to do this?