Design comparison
SolutionDesign
Solution retrospective
I would like some feedback on CSS. I feel that my css is a bit messy(?). Probably should work on organize and name the class in readable/clear way.
Community feedback
- @MurkragePosted over 3 years ago
I like that you've used CSS variables! Naming wise you could simplify things for yourself by renaming
bg-pattern-card
toprofile-pic
. They are both used on that element and it'll make more sense when reading the stylesheet. First you'll have.profile-pic
, which has the background image, and then you'll have.profile-pic img
which targets the actual image..profile-pic { background-image: url("./images/bg-pattern-card.svg"); } .profile-pic img { border: 6px solid white; border-radius: 50%; transform: translateY(50%); }
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