@hayam4k3r
Alexsandro Agustini Junior
@AlexAgustiniAll comments
- @AlexAgustini
Cool! Tip for centering the whole container: in cases like that, just put in on the body: min-height:100vh; display:grid; place-content:center; and that should do it
Marked as helpful - @Owura-56@AlexAgustini
Hey man, nice project, just a little tip: in the "perfume" heading at the top of the card, you could add a "letter-spacing" property to space the letters, i guess it would be like 1 or 2 px, so it gets closer to the original design
- @Roneeey@AlexAgustini
Hey man, good code, but i spotted a problem: although your site works in 1440 and 375px, it breaks in between this widths, in like tablet widths, so to fix that, you can try to give your container a "width:min(90vw, 1000px)" for example, so that below your container occupies 90vw of the screen, but have a maximum width of 1000px, which would help on bigger screens so the text don't get too stretchy
Marked as helpful
