Design comparison
Solution retrospective
Hello all, this is the first challenge I submit and any feedback would be very much appreciated!
Community feedback
- @MasterDev333Posted almost 4 years ago
Great work @Maria. You used pseudo-elements to draw background circles. It was a nice try. It would be better if you set over-flow: hidden to body and width: 100vw, height:100vh to hide scrollbars. Here's my solution if you'd like to check. https://www.frontendmentor.io/solutions/profile-card-xRCu-QVRG Happy coding~ :)
0@MsglPosted almost 4 years agoThank you for the feedback @MasterDev333 ! I will try out the solution you suggested and check your code as well!
0 - @janegcaPosted almost 4 years ago
Hi Maria, that looks pretty good for your first challenge. Only a minor suggestion, try setting your body width and height to viewport width and height so things don't stretch quite as much. Nice job :)
0@MsglPosted almost 4 years agoHi @janegca thank you for the suggestion, I will try it out!
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