Angelo
@AngeloMar98All comments
- @sakshmm347Submitted over 1 year ago@AngeloMar98Posted over 1 year ago
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-RicardoSubmitted over 1 year ago@AngeloMar98Posted over 1 year ago
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 - @noneofurbuzzSubmitted over 1 year ago
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.
@AngeloMar98Posted over 1 year agoI 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 helpful0