raya• 2,850
@rayaatta
Posted
Hello 👋chirag-bishnoi, congratulations on completing your first FED challenge 🎉
I have some suggestions for you.
1
Since the .profile image is not decorative you should fill its alt attribute with the name of the person on the image I e
Alt="Greg Hooper"
2
I noticed <div class="wrapper"> replace it with <section class="wrapper"> in order to make your html more semantic.
divs do not carry any semantic value.
They are used solely for decorations.
You should replace some of them with tags such as <article> and <section>
You can find out more about these tags and more in this article📑.
I hope this helps 🙃
Otherwise nice solution 👏
Marked as helpful
1
