@ysmltr
Latest comments
- @ninjabro
need slight attention for the padding in the text container and parent card width. looks amazing!
Marked as helpful - @Natrosvili@ninjabro
congrats on completing this challenge, did you think of changing grid properties for the mobile screen size?
@media screen and (max-width: 375px) { .gridbox { grid-template-columns: 1fr; }
i assume this might let the containers align on top of each other.
- @sawsan-code@ninjabro
did you think of putting those annual plan and $59.99 into a div?
make them as flex container and use flex direction to column.
- @christinnaa@ninjabro
hey, may i ask you how did you get the exact size of card? did you use ruler or something? thank you.
- @rymxoxo@ninjabro
i might want you to work on the layout first
size for desktop is too big, reduce the size and you should be good to go!
i used grid for layout and flex for inside div containers
you could improve these below listed
center the card container in body , paddings around the perfume card, button font size, height of button.
hope this might help you
