@Sujood-Qureshi
Posted
Hey Harshit! 🙋♂️
Congratulations on completing the Profile card component challenge
I have some suggestions for you about your code.
Index.html
-
Use <main> tag to wrap the page's main content instead of <div> tag. It will show the search engines that the particular content is essential on that page. You can read more about <main> tag here.
-
You should use the <h1> tag. it is a best practice to ensure that the beginning of a page's main content starts with <h1>tag element, and also to ensure that the page contains only one <h1>tag element
Style.css
- Use relative units like em or rem instead of the fixed unit like px. The relative units can change elements size to the parent container
I hope you find it useful! May your front-end journey be filled with successful 😊😊
Marked as helpful