Design comparison
Solution retrospective
I used BEM for naming classes.
What challenges did you encounter, and how did you overcome them?I encountered difficulties with the image shrinking due to margins reducing the available width, which affected its aspect ratio. I resolved this by using a fixed width for the image.
What specific areas of your project would you like help with?I would appreciate advice on more effective methods for addressing image shrinking and spacing issues within a flexbox layout. Any suggestions for improving responsiveness or optimizing the layout would be helpful.
Community feedback
- @JawadM2002Posted about 2 months ago
The solution provided shows a good mixture of HTML and CSS, particularly with how it is able to follow the desktop design as expected, and moreover, you have demonstrated a good use of CSS variables that helps with the theme colours. The layout looks very good and has very well-structured code. You've also made good use of the media query using the min-width function so that the user is able to determine what the width for the page would be. Overall, the solution looks very identical to the design, but is able to follow the expected outcome and result, so very good and keep it up.
0 - @oduwa-APosted about 2 months ago
I don't really have much critique for this since it was well made. Maybe invest into object-fit if possible for image spacing, but otherwise I don't know. Good job however.
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