Design comparison
Solution retrospective
Any feedback would be appreciated!
Community feedback
- @Aik-202Posted about 2 years ago
Hi, Khan... Nice work!!!!. I have some suggestions though :
-
Try changing the
section
tag with class maincontainer tomain
tag, as every html page must have a main tag, as it helps screen readers know where the main content of the page is located. -
Remove the the href attributes that have no value from the anchor tags, that should sole some of your html issues.
-
For the mobile mode, it's not exactly pixel perfect, you can also reduce the size of the images as they appear too big. You can also reduce the font-sizes of the headings, they also appear too big.
Hope this helps.
Marked as helpful0@Khanza2Posted about 2 years ago@Aik-202 Thank you for your valuable feedback Chioma. Yes, I've been working with my responsive design and I know It kinda looks not good, but anyways, I'll consider your feedback, thanks!
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