@emestabillo
Posted
Hey @trafiki, great work! Project is responsive and close to the design.
-
I would apply
min-height: 100vh
to themain
tag so that the component spans the entire viewport height, and centers the component on the page (with flex). This way, you don't need the top and bottom margins on.order-card
. The user will also be able to scroll down if the contents doesn't fit their viewport, better thanheight: 100vh
-
The alt text
header-image
doesn't convey meaningful content. I might just leave the alt attribute empty or use the image as a background -
Good use of interactive elements. I would probably use an anchor tag for 'Proceed' since it usually brings the user to another page to complete payment
Hope this helps!
Marked as helpful
@trafiki
Posted
Thank you so much @emestabillo. Thanks for min-height hack, I've always wanted that kind of behavior. This was really helpful