Design comparison
SolutionDesign
Solution retrospective
Feedback on this will be great and any feedback on how i can improve this solution will really be helpful. any feedback is appreciated thank you all guys
Community feedback
- @shashreesamuelPosted over 2 years ago
Good job on this challenge. Keep up the good work.
Try removing the
div class=`attribution`>
to fix the accessibility issueMarked as helpful1 - @Yemisrach15Posted over 2 years ago
Hey Kasemkl, The mobile design is not responsive. I have to slide sideways to see the content. Here's how I suggest you correct it.
- Remove
min-width: 100vh
frombody
. I don't think you should be usingvh
for width. Usevw
- Give
width
tobody
likewidth: 87%
- Set
margin: 0 auto
to center it.
Marked as helpful1 - Remove
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