Design comparison
Solution retrospective
This is my first project and also my first time using SASS. Any feedback would be very appreciated.
Community feedback
- @Rabin92Posted almost 4 years ago
Hello @munloop,
Congratulations on completing your first challenge!
You have done a great job here. The design almost matches the mockup and looks great on most of the devices. My only suggestion is on a small screen landscape mode, the
attribution
border touches the card, to fix this problem you could simply addmargin-top: 20px
to theattribution
class.Your
Sass
is looking good too, you have grasped a good understanding of features likevariables
andampersand
. To learn more about other features you can visit this awesome website: SassHappy Coding!
0@munloopPosted almost 4 years agoHi @Rabin92,
Thank you for the feedback, It was really helpful and I was able to fix the issue with the attribution class.
Happy Codding!
0
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