Design comparison
Solution retrospective
For some reason I did not like my solution.
I guess it has something to do with the positioning of the profile picture.
Any advice on how to improve my solution?
Community feedback
- @VCaramesPosted about 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
-
The purpose of the Main Element is to identify the main content of your page. It is not the container of you component. After the main element, you want add a container to wrap you separate components in.
-
The background images should be applied using CSS not HTML. You can accomplish this by using the
background-image
property. -
The profile card’s background is decorative, it does not add any value. So its Alt Tag should be left blank and have an aria-hidden=“true” to hides them from assistive technology.
-
The only heading in this challenge is “Victor Crest”. Everything else should be wrapped in a Paragraph Element.
-
The statistics are a list. They need be created using the Unordered List along with the List Element.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful0@joaovitorwittPosted about 2 years ago@vcarames Thanks for the feedback. I am currently working on that
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