@KapteynUniverse
Posted
Hey Danny, nice job. Small things;
I think you can use article tag for the card-wrap div. The article tag specifies independent, self-contained content. An article should make sense on its own and it should be possible to distribute it independently from the rest of the site.
On the body you can use min-height:100vh instead of 100%, on content wrap justify-content: space-between; looks better than the center imo.
Gabrielle Essence Eau De Parfum could be in a h2 instead of p tag. On a full web page there will be h1 somewhere, so that is why i said h2 for this.
Also use a modern css reset for future projects
Oh forgot to mention. Using classes for styling would be better. ID is specific to a single element, classes can be assigned to multiple elements on a page or throughout the website.
Marked as helpful
