sakshmm347• 50
@sakshmm347
Submitted
Angelo
@AngeloMar98
All comments
- Angelo• 490
@AngeloMar98
Posted
This is a really beatiful solution but you uploaded it in the wrong section, this is for the calculator app, which you already did too
0 - Leone DaVinci• 340
@Leone-Ricardo
Submitted
Angelo• 490@AngeloMar98
Posted
The image messes up the page a bit, I would suggest using max-width to handle it or leave it at width 100% but doing something with the container. You handled the change to the mobile img as I did, but the media query in the source element needs to be between parenthesis
1 - Ugochi Ike• 180
@noneofurbuzz
Submitted
I feel like there could have been a better to implement this project. Any ideas? Also i don't like how it looks on devices with a large viewport but I didn't know how to optimize the design to fix that.
Angelo• 490@AngeloMar98
Posted
I really like this solution, regarding the card image, you opted to use pixels for everything, which is a good idea since it's not that important for the font to scale there. And I'm really surprised by how the input bordered is made on focus, I didn't even know background had those options! One thing, which is very small, center the credit card number more to the right for desktop, other than that this solution is better than mine in every way, I really liked it
Marked as helpful
0
