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.
@pikapikamart
Posted
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 helpful
@seniorteck
Posted
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 helpful
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
