Design comparison
SolutionDesign
Solution retrospective
Just Built a stats preview card using HTML and CSS only.
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, awesome job on this one. Layout in desktop is wider than the original, on the responsiveness, it scales but at some point, the screen starts to hide the layout sides, try inspect it and change the screen's width. The mobile state however looks good.
Some suggestions would be:
- Have a look at if your font-family is applying properly, right now it is not the same as the original.
- For the
info_content
selector, you could have usedul
element on it, since those are "list" of information about the company website. - The text inside the
info_content
shouldn't be a heading tag as well. If a text is bold in a design, it doesn't always mean that they should be heading tags. - The
alt
for the image on the left side could be improved. Also when usingalt
, avoid using words that relates to "graphic" like "image, picture, icon...", it is already an image, no need to describe it as one. - Lastly, just the responsiveness, check out what I said about the layout being cut off.
Aside from those, great job again on this one.
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