2Responsive default using double background and positioning
Design comparison
Solution retrospective
I've learned a lot doing this challenge, i'll be glad if there's any feedback :D
Community feedback
- @vanzasetiaPosted over 2 years ago
Hi, Kimue! 👋
It's great to see you submitting another solution! Good effort on this challenge! 👍
The alternative text for the profile picture should be the name of the person and also, keep in mind that it's already an image element so there's no need to specify it within the alternative text.
For the stats, I recommend making it a list because if the site has no styling, it would more likely be a list.
- 80K Followers
- 803K Likes
- 1.4K Photos
In addition to that, it's best to write the HTML first without any styling before writing any CSS. This way, it can help you to choose a better HTML element.
I would strongly recommend making the
body
element as the main container and usingmin-height
instead ofheight
. Also, remove thewidth
property because by default block element has100%
of width.It's best to let the content inside it dictate the
height
of the card. This way, if the content changes, you don't remember to change theheight
of the card.That's it! Happy coding! 😁
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