Profile Card Component

Community feedback

• Account deleted

  Hi there! Congrats on completing this challenge!! 🥳

  Your solution is good, but it can be improved.

  HTML:

  Your blue image background, You shouldn't have alt text, because it doesn't add value to your page, it's just decorative. The correct way would be: <img src="./images/bg-pattern-card.svg" alt="" class="cover-image">.

  Header elements implements six levels h1-h6, with h1 being the most important and h6 being the least important. avoid omitting header levels, your page should start with the header h1 and successively with h2...h6. More info here.

  CSS:

  Your styles are in your index.html, that's ok, but the correct way would be to have it in another file, in this case styles.css

  I didn't see a reset in your css file, I recommend you use it, using it will save you future headaches when designing your page. Here some resets:

  • Normalize.css
  • Reset.css
  • My Custom CSS Reset of Josh Comeau

  I hope I've helped! 🙃

  Happy coding! ✌️

  Marked as helpful

  1