Design comparison
SolutionDesign
Solution retrospective
Provide your solution link if you done it. Thanks
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, awesome job on this one. Layout in desktop is good, responds well and the mobile layout is good as well.
I haven't made this challenge so can't provide any link, but for suggestions, here are some:
- Always have a
main
element on a webpage. Usemain
to wrap the main content of the page, this helps users to properly navigate your website. - For the
card__features
section, it would be great if you usedul
element on those, because those are "list" of company website informations. - The
card__feature-title
shouldn't be a heading tag, because what good does10k
gives if it is a heading tag, right? It doesn't really give any extra information or be suited to be a heading tag. In design, it doesn't mean that when a text is "bold", it should be a heading tag.
Just those little suggestions, really great job on this one again,
0 - Always have a
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