Design comparison
Solution retrospective
feed backs are welcome
Community feedback
- @law973Posted 2 months ago
Hey! I like that the profile is viewable on the full range of screen sizes.
I'd suggest using anchor elements in place of the h4 elements, for a couple of reasons. Those parts of the card are intended to lead to Jessica's profile pages, so using hyperlinks would be a good choice for that. Also, it would be best to avoid skipping heading numbers if possible; going from h1 to h2 to h3 would be better than going from h1 straight to h3, for example. If an h2 needs to be smaller like an h4, it can be resized with styling.
Using pseudo-classes such as :hover would allow the background color and text color of the links to change upon user interaction, and would help in creating a responsive and complete design.
Marked as helpful0 - @ggvictorPosted 2 months ago
I think you should have put more of your face in this project, like your photo and your profile links, you also forgot to put a pointer cursor on the button and put a hover with css too
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