Social Proof Section solution using Flexbox.
Design comparison
Solution retrospective
Updated solution. Fixed the body background images in lower resolutions and other semantic issues.
Community feedback
- @guztrilloPosted about 3 years ago
Hey Joãor, great solution!!! I love how you played with the layout for multiple screens. It looks very cool and clean.
Marked as helpful1@JSaporskiPosted about 3 years ago@guztrillo thanks for the feedback. :D
0 - @dusanlukic404Posted about 3 years ago
Hi Joao, well done on this challenge! 🎉 It looks same as design. 🔥 In my opinion you should only make <h1> smaller on mobile version. Also, you should increase your heading only by one so use <h2> instead of <h3>. 🧐😃
Marked as helpful1@JSaporskiPosted about 3 years ago@dusanlukic404 thanks for the feedback man, I've just fixed these issues. :D
0 - @dewslysePosted about 3 years ago
Hello @JSaporski! Congrats on submission. Your page looks very nice 🔥🔥. I like your use of flex-wrap. It gives the tablet view a very nice look. Good job 👏
Marked as helpful1@JSaporskiPosted about 3 years ago@dewslyse thanks man 🙂, the only thing missing were the two background images of the page at lower resolutions, i'm still learning about the "background" properties, especially the "background-position", but i'll fix it soon.
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