Design comparison
Solution retrospective
I was stuck to make the background image with responsive design. 😭 Is there any better way to write this code with two background-images?
Community feedback
- @vanzasetiaPosted over 2 years ago
Hi, Pon Huang! 👋
Congratulations on completing this challenge! 🎉
Yeah, positioning the background images is the hardest part of this challenge. For this, I would recommend looking at the @ApplePieGiraffe's solution. He had done a great job of positioning the background images. He managed to make all the circles position correctly on all screen sizes.
Some suggestions from me.
- For the
profile-state
, I would recommend making it a list element. If the site has no styling, it would more likely be a list.- 80K followers
- 803K likes
- 1.4K photos
- I recommend adding
rel="noopener"
to any anchor tags that havetarget="_blank"
. This is a security essential for external links. I suggest reading the web.dev article to learn more about this.
Hope this helps. 🙂
2@ponhuangPosted over 2 years ago@vanzasetia Thanks for sharing ApplePieGiraffe's solution, it is amazing. And also the suggestions help a lot 😃 Thank you
0 - For the
- Account deleted
I'm not sure about the background but I see you're having issues with centering the content so that it's in the middle of the screen, at least on the desktop. If you set a height of 100vh (100% of viewport height) on the body element the cards will be fully centered. Currently your body elements are too short to see the effect of flexbox.
1
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