Submitted over 3 years ago
Four Card Feature Section built with HTML and CSS
@Johndiddles
Design comparison
SolutionDesign
Solution retrospective
Please, check out my submission on this challenge. I'd love your feedback. My code may look horrible 😁, feel free to stone me. Is there anything I could have done better? Please, let me know. 😘
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hello there, John_diddles! 👋
Nice job on this challenge! 👏
I suggest,
- Using a single
<h1>
tag for the heading of the page (since it is really a single heading, not two separate headings) and it is recommended that you only use one<h1>
tag per page. You can easily wrap the different parts of the heading in<span>
tags to style them differently. - Adding another media query or two to make sure that your site is responsive for all screen sizes. Currently, a horizontal scroll bar appears along the bottom of the page at around 950px before the layout changes from desktop to tablet/mobile.
- Increasing the size of the text in the feature cards just a bit (they look a little small, at the moment).
- Increasing the font-weight of the second part of the heading (as in the original design) to make it bolder.
Hope those tips help. 🙂
Keep coding (and happy coding, too)! 😁
Marked as helpful1@JohndiddlesPosted over 3 years ago@ApplePieGiraffe Thanks. This is more than just helpful. 🤗
1 - Using a single
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