Submitted over 4 years ago
HTML, CSS, & JS validation / mobile first / tablet & desktop sizes
@KuroiwaAiki
Design comparison
SolutionDesign
Solution retrospective
Feedback appreciated! Thx!
Community feedback
- @adarshcodesPosted over 4 years ago
Hi! @KuroiwaAiki, Great work👍 Let's take a look at your solution:
- Design aspect: your solution is spot on👏 and looks nice. But there is a problem on the bigger screen there is a space left on the right side of the image. Also when it hit the breakpoint 1000px the image is not covering full width due to the container size which is 70%.
- Responsive aspect: Responsiveness works fine, just fix the image width if it's not intended.
- Issues aroused: Good job, no issues. Happy Coding🙂
2@KuroiwaAikiPosted over 4 years agoThanks adarshcodes! The max width 1440px is intended; however I can change it for larger screen resolutions.
The 70% width is also intended for tablet sizes as I didn't like the way the picture stretched when it was full width.
Thanks for your feedback 🙂
0 - @juandadevPosted over 4 years ago
Look the same between your solution and the design preview, amazing!
1
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