@pradeeps4ini
Posted
@romenter. Well done on completing the project.
I would like to suggest some changes, if you don't mind.
-
Use semantic markup in HTML. Semantic markup is better for readability and accessability. General container could have been <main> and sub containers could have been <section>.Semantic Markup
-
To use different size images on different viewport widths, use <picture> element.
-
Your header image should have a right margin. Use margin-right or margin-inline-start to give some margin to the image.
-
You can create an utility class. .flex {display: flex}. And use it on the elements that need to have ,"display: flex;" property. This way you don't have to re-write same code in css.
Marked as helpful
@romenter
Posted
@pradeeps4ini That's exactly what i was talking about when asking for tips. All noted. Tkn u very much 😀
@pradeeps4ini
Posted
@romenter You're welcome dude. Have fun learning and implementing more projects.