@Harsh-Kumar-Dwivedi
Posted
Hi @bhuvandev16,
Nice Work !
I would suggest following:-
1.Prefer replacing <div class="container">
with <main>
element.
2.Prefer using <h1>
instead of <p>
in the following and then adjust it's styling in CSS
<p class="heading">Improve your front-end skills by building projects</p>
3.Prefer replacing <div class="Attribution">
with <footer>
element.
These would improve accessibility and make your markup more semantic.
Hope these suggestions are helpful !
Happy Coding !
Marked as helpful
@bhuvandev16
Posted
@Harsh-Kumar-Dwivedi Okay, I will implement this in my upcoming projects. Thanks for your suggestions man
@Harsh-Kumar-Dwivedi
Posted
@bhuvandev16
Welcome !
Keep up the good work !
If you find my suggestions helpful, kindly, please consider marking them as helpful.