Design comparison
SolutionDesign
Solution retrospective
I would be really happy for any feedback as that is how I am learning. Thank you.
Community feedback
- @kjdeepakPosted over 2 years ago
- Remove unnecessary text.
- Check for mobile responsiveness
- Use a CSS methodology while using vanilla CSS for your project so this will help you in maintaining styling rules in complex projects.
- Check for word breaks and font size.
Marked as helpful0@zambobencePosted over 2 years ago@kjdeepak Thank you for your feedback I will take them into consideration in my future projects. Which CSS Methodology would you suggest me to start with?
0@kjdeepakPosted over 2 years ago@benceturbulence You are welcome. You can consider going through this documentation http://getbem.com/introduction/
Marked as helpful0 - @antonio-lopezPosted over 2 years ago
The layout looks good on layouts larger than mobile. On mobile your text is too big and not aligned with the design. Adding some media queries that target the text and image on smaller screens can 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