Four Card Challenge using Flexbox and SCSS

Solution retrospective
Feedback always welcome :) This was my first time trying SCSS and I think its amazing. It definitely sped up my workflow so I'll be using it a lot more in the future.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @kenreibman
It looks great!
I commend you for implementing practices such as SASS early.
Although Frontend Mentor doesn't specify, I would personally create a tablet breakpoint
~700-1000px
, and rearrange it for a tablet view, since the containers look too stretched around that viewport width. Here's what I did: siteI would also put
aria-hidden="true"
and not have analt
tag for the icon images, as they really don't serve any purpose for someone accessing your website who cannot see the icons.alt
tags should be used to describe an image that's important on a website, like describing the product you are selling on an image.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