Profile card component

Community feedback

• 

  BrunoMoleta•700

  @brunomoleta

  Posted about 2 years ago

  Regarding accessibility, according to MDN:

  • <h1> is the element that describes the content of the page;

  That means the h1 of our page shouldn't be the user's name but rather the Profile card of Victor Crest, which is more helpful for screen-readers.

  Also, it's a good call to start the bottom part of the card with an h2 with User statistics above it so it gives a good impression about the content to come next. Like <h2>Profile Statistics</h2>.

  And then, it would be necessary to hide them in the CSS using: h1, h2 { display: none }

  I learned that reading the code from Melvin Aguilar, one of the best-rated devs in the community.

  Apart from that, your solution looks fine, Andreas.

  Bis später :)

  02

  Marked as helpful