Notifications page solution - not sure about the sementics

Community feedback

• 

  Catalin Banici•260

  @CatalinBaniciPosted about 2 years ago

  You should use an <ul> tag for the container and each notification inside an <li> tag. The avatars and every link should be in a empty anchor <a> tag, empty, because there is nothing behind them. The private message also should be in a different element tag, not sure which one, but in my solution I used <aside>, just to be different from the others.

  edit: I just noticed, you did put the links in <a> tags, but you should also put the <img> tags from the avatars, usually you can click either on the user's avatar or user's name to see its profile in most cases, I guess...

  Marked as helpful

  1

  ---

  

  ~ Ammar•100

  @andernaderPosted about 2 years ago

  @CatalinBanici Thanks alot.

  0