Design comparison
Solution retrospective
I am really pleased with the responsiveness using grid and media queries for the layout. My first time applying SASS to a project so it took me much longer to complete this. Also first time doing a mobile-first design so having to think backwards was challenging. I was going to use a pseudo element (::after
) for the "old price" but I realised this was not good for accessibility.
I had difficulties getting my head around using breakpoints, using clamp()
for responsive fonts and width: min()
for responsive layout. Did a lot of trial and error before finding the sweet spots. Also the architecture of SASS was a bit confusing at first, knowing how to structure the folders. I can see now how this makes it easier to work on a project.
Also getting the image to change from mobile to desktop took me a while, looking at StackOverflow I realised I could include both images in the HTML, using display: none
or display: block
at the right breakpoints allowed me to hide and show the relevant image.
I could probably structure the SASS folders better next time, it's new to me so it's a learning process at this point. Any feedback on whether I could have done anything differently to make the code simpler would be appreciated.
Community feedback
- @dylan-dot-cPosted about 2 hours ago
Well done! This is a great solution.
One thing you can do differently is to use the
picture
tag/element to easily switch between pictures by using the media attribute to specify what ranges this img should show on. I suggest you search it up and see how it can be used.Also for
item-pricing
you can use flex instead of grid just for less code and complexity.1
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