Design comparison
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.
Community feedback
- @kenreibmanPosted over 2 years ago
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 helpful1
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