Design comparison
Community feedback
- @Ryan-OHanlonPosted 4 months ago
This is an impressive use of only using CSS Flexbox and making breakpoints for both mobile and desktop. I would highly recommend taking the time to learn how to use CSS Grid as it can help you with placement of the container elements to be aligned.
Your code is structured well separating each testimony container in its own class but it can be improved as some testimonies are nested in each other in your HTML when they could be a child of the bigbox class and not the longbox class. Limiting yourself to just CSS Flexbox will only make placing complex elements like these more difficult in the future. Which I see with Kira's testimony being mis-shapen when I adjust the width of the browser that causes the margin and padding to fall when the display goes past 1000px.
Remember that when you need to place the entire body element in the center of the webpage to assign it the CSS attribute min-height with a value of 100vh.
You'll be using CSS Grid and Flexbox together for future projects to obtain that responsive web design. Here's a site that helped me out understand grid. https://gridbyexample.com/examples/ https://www.joshwcomeau.com/css/interactive-guide-to-grid/
Marked as helpful0
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