Design comparison
SolutionDesign
Solution retrospective
to the one who will review this code, thank you. i am hoping for your feedback on how can i improve my skills.
Community feedback
- @NikitossikPosted over 1 year ago
Hey, good job)
Here are some improvments I'd like to tell you:
- instead of adding 2
<img>
tags, you can use <picture> to change the image on the certain breakpoint if needed - when you flip the content on the mobile version, you could change a
@media
query to max-width: 700px (card width), because a horizontal scrollbar appears between the breakpoints. Or you could changewidth: 700px
tomax-width: 700px
, just to avoid fixed width, because it can cause issues
Good luck)
0@eadq001Posted over 1 year ago@Nikitossik thanks for your review, i'll keep that in mind.
0 - instead of adding 2
- @MohamedRiya-progPosted over 1 year ago
-Could have aligned the button to the center. Product description font weight could have been adjusted too.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord