Design comparison
SolutionDesign
Community feedback
- @tobaojoPosted 23 days ago
This looks really good, the hover states work well. Also looks like you managed to figure out how to incorporate the fonts
some pointers:
- The
border: 4px black;
in.wrapper
is missing the solid keyword. Adding "solid" will ensure a consistent border style. (e.g.1px black solid
) for the thickness, colour and style of border - In your h1 styling, you have duplicate transition declarations. You only need one.
.container .imagess
can be confusing due to the double “s”. Renaming it to something like.image-content
- You have a typo with
lin
(line 124 in your css) in the.tag
class. you can remove any unused or incomplete styles. - lastly for your
font-weight
U I think you need to remove the px so it's just a number (e.g.font-weight: 200
)
Great effort all around!
Marked as helpful0 - The
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