Responsive Site using box-sizing and flex box - Four card preview
Design comparison
Solution retrospective
Hello Everyone!
If you notice some error in my code please give a feedback on what I did wrong.
Other than that any suggestions on how to improve my code will be appreciated.
Thank You : )
Community feedback
- @palgrammingPosted over 3 years ago
🌟🌟🌟🌟🌟 Great Job. Wonderful layout transitions and your card content looks great in all stages of transition
1 - @tedikoPosted over 3 years ago
Hello, Saurabh Mishra! 👋
Well done on this challenge! Here's a few things I'd suggest you to take a look at:
- You should wrap your main heading into one heading element instead of using two. If you want to style some text within that heading other way just wrap that part into
<span>
element. Like this -<h1>Reliable, efficient delivery <span>Powered by Technology</span></h1>
Keep up the good work! 💪
0@Saurabh-Mishra04Posted over 3 years agoHello @tediko! Thanks a lot ! Wish I had thought about it. It can actually save a lot of time because in my code I had to adjust a lot of properties like the margins, paddings , colors etc as they were different. But with this it makes the job easier! Next time I will use your method for sure.
Thanks for the appreciation and will keep up the good work .
0 - You should wrap your main heading into one heading element instead of using two. If you want to style some text within that heading other way just wrap that part into
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