Marlon Passos• 940
@MarlonPassos-git
Posted
Good job,
some suggestions
- "Proceed to Payment" could be a <a> tag
- It would decrease the font size a little
- Increase the size of the card a little
- change the repeat to
background-repeat: repeat-x;
because in this way the background breaks on big screens - Add a hover effect to "cancel" and "change"
- Change your card padding to
clamp(1.25rem, 0.8099rem + 1.8779vw, 2.5rem);
, this would automatically change the value from 20 to 40px flexibly and without media query
Marked as helpful
1
Ferd• 30
@FerdieM
Posted
@MarlonPassos-git Thank you for your feedback..
1