Mukarram Haq• 230
@MukarramHaq
Posted
Hey, great work!
I would like to point out a few things:
HTML:
- In your
<body>put everything inside the<main>tag for this particular project. It specifies the main content of the page. - Always have something in you
altfor an image. You can say something like Chanel Parfum <br>tags should be avoided write the whole sentence in one line.- In your
<button>tag you can give the source of the SVG that you downloaded with this project it should beimages/icon-cart.svg
CSS:
- In
.imagenever use px for width. Always yourrem. - The
font-sizeshould never be inpxuseremfor that as well - In
buttonremove the borders byborder:none.
This is all I could help you with with my limited knowledge. I hope you find this useful :)
0
