Design comparison
SolutionDesign
Solution retrospective
Any feedback would be great. Thank you.
Community feedback
- @mattstuddertPosted about 5 years ago
Great work on this challenge Tish and congrats on submitting your first solution! Here are a couple of pointers after taking a look at your code:
- You've currently got no headings on the page. I'd recommend using them to create a content hierarchy. This is important for accessibility and also the SEO of any website.
- I would also recommend never using IDs for styling purposes. They're too specific and can't be reused on the same page. Instead, stick to class, attribute, pseudo, and type selectors. Doing this will keep your CSS code more maintainable as it grows.
I hope you enjoyed the challenge. Keep up the great work! 🙂
0@TishGPosted about 5 years ago@mattstuddert Thank you, I will go ahead and implement some of those changes you suggested.
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