Agustina• 125
@aguscorvo
Posted
Congrats on solving the challenge! Here are some suggestions or tips:
- Include semantic HTML tags like main, section, and article when necessary, instead of using divs for everything.
- There’s something missing in the hover state: while the background color changes, the text color stays white, which makes it hard to read. According to the designs, it should be black.
- From my understanding, "GitHub", "Frontend Mentor", etc, should be buttons.
- Explore other ways to manage spacing between elements instead of using <br> tags. For example, you can use the gap property with flexbox.
- Please check the size of the avatar image, the card size and the padding. Also, note that they may change slightly in mobile view.
Happy coding! :)
0
