Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive four card page with HTML CSS (Flexbox-Grid)

@ToprakPehleeone

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I had little knowlage about grid before this challenge. After watching tutorials and reading documents about grid, i was able to find a solution for desktop and mobile designs.

What challenges did you encounter, and how did you overcome them?

Aligning containers like a diamond using grid was a challenge. I used 'grid area' properity to find a solution.

What specific areas of your project would you like help with?

Any suggestions and feedbacks on any topics are very much appreciated! Thanks.

Community feedback

@TedJenkler

Posted

Hi @ToprakPehleeone,

Nice project! Here are a couple of suggestions to improve it further:

Heading Hierarchy: I noticed a bit of an issue with your <h1>, <h2>, and <h3> tags. Remember that headings should be used in order of importance, with only one <h1> per page. This structure helps screen readers and Google’s search engine better understand your site.

Avoid Excessive Divs: Try to avoid using <div>s when possible. Instead, use more semantic elements like <article> or structure your cards and components with Flexbox for cleaner, more maintainable code.

Responsiveness: Your page isn’t responsive yet. I highly recommend using Chrome DevTools to check how your site looks across different screen widths (from 320px to 1440px). A mobile-first approach, using rem or em for units, will make responsiveness easier to manage. Also, avoid setting fixed heights—use min/max-widths for better flexibility.

Keep up the good work!

Best, Teodor

Marked as helpful

0

@ToprakPehleeone

Posted

@TedJenkler

Thank you so much for very helpful feedback! I will update the code according to your suggestions. :)

1
MikDra1 6,050

@MikDra1

Posted

Nice one 😀

If you are curious how you can do this straight lines on the top of each card here is my tip:

Create another element in each of the cards. Then position this element absolute. Card should be positioned relative. At the end you need to give this element a height of 3px width of 100% and top 0 and left 0. You can also use ::after or ::before pseudo elements to create these.

Hope you found this comment helpful 💗

Good job and keep going 😁😊😉

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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