Skip to content
Submitted almost 3 years ago

Responsive layout with Sass

accessibility, sass/scss
Daniel Chyliński•160
@danielosd68
A solution to the Four card feature section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I think this project is the best project I've ever done. 😁

Code
Loading...

Please log in to post a comment

Log in

Community feedback

  • Account deleted

    Instead of Flexbox I recommend using Grid. That way you won't have to group the middle two cards in one div. Here's a great tutorial for Grid.

    Marked as helpful
  • mubizzy•1,520
    @mubizzy

    Excellent job on this challenge! your report has a few issues though:

    1. wrap everything in your body in <main> or use semantics

    2. it is a best practice to use both HTML 5 and ARIA landmarks to ensure all content is contained within a navigational region.

    Hope it helps:)...don't forget to mark it as helpful 👍

    Marked as helpful
  • Adam M•550
    @AdamMzkr

    Really nice solution! Only what you can change is add semantic tags instead div's ;)

    Marked as helpful

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