Design comparison
Solution retrospective
It is my first time using Sass and any feedback on how to improve on this project or do something completely different is welcome. Thank you for your valuable feedback on this project.
Community feedback
- @pikapikamartPosted over 3 years ago
and may I suggest for the queries, for the mobile state, try making at width from 350 to 450px at that range for phone, since phones nowadays have their ppi at that range, and I saw that your mobile max is at 375px and somehow will not support devices at that upper range.
Marked as helpful0 - @seniorteckPosted over 3 years ago
hello paul, these are just my suggestion though on ways i think you can improve your code 1 - for the media queries use px instead of the em 2 - the word spacing for the p tag i think you should reduce them for the mobile view 3- on desktop media query you can increase the width for the content. and try go through the errors indicate on the html flag by front-end reports guide.
also the link to the github is broken. you can try fix that
Marked as helpful0
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