Hanana• 170
@meantoes
Posted
hi! congrats on completing the challenge⭐ I have some suggestions that might interest you,
-
wrap the whole content in the <main> tag instead of the div
.container
-
use alt attribute for the perfume image
for the css, use this trick to center the container
- add these properties to your
body
element
body {
/* -- */
display: flex;
align-items: center;
min-height: 100vh
}
- in the
.container
element, deletemargin-top: 100px
- Use relative units like
em
orrem
for margin, padding, width, and height. here are some sources about CSS units, Relative and Absolute units in CSS and more complex source
hope this helps
Marked as helpful
1