@efecollins
Posted
Hello Yashvir Singh, great work you've done here. Thumbs up!
In addition to the corrections @correlucas pointed out. These are some things you could do to fix your project's accessibility and HTML validation issues.
Accessibility
- Add the attribute
role="main"
to<div class="profile">
. This makes your document have one main landmark.
HTML Validations
-
For your images,
bg-pattern-card.svg
andimage-victor.jpg
. Use your internal CSS for the styling, rather than inline. The syntax you used is deprecated. -
Change this
<hr width="100%" height="1px" style="margin-top: 8%;">
to<hr style="margin-top: 8%;height: 1px; width: 100%;">
I hope this helps and I wish you a happy coding journey!