Submitted almost 3 years ago

Services cards layout using CSS Grid

ChrisEski• 110

@ChrisEski

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Four card feature section

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

When switching to desktop layout, and after setting css grid, the red and orange cards that are together in the central column, had a smaller height than the other two on the sides.

Min or max height on these cards had no effect, they kept staying shorter. What worked was setting the grid row height to the given height, as well as setting the cards' height explicitly, which is what i sticked with.

Is that a correct/good approach? Was there another way of making the cards' height go up to 250px without explicitly setting to this number?

Community feedback

David Turner• 4,150

@brodiewebdt

Posted almost 3 years ago

This came out fine. There is no right or wrong way to code a site. Everybody's code will be slightly different. You should make the main section without any other classes. Just create another div inside the main section. If you do that it will clear the issues in the report. I downloaded the AXE DevTools and use that to check accessibility: https://www.deque.com/axe/devtools/

Marked as helpful

ChrisEski• 110

@ChrisEski

Posted almost 3 years ago

@brodiewebdt Wow that tool seems super useful, just like your comment! Thank you so much!

David Turner• 4,150

@brodiewebdt

Posted almost 3 years ago

@ChrisEski It is, it will say that there is low contrast in some of the designs, but that will still pass. That is the design not the coding.

Please log in to post a comment

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