@katjadanilova
Posted
Hi, Nyrell!
Well done! I liked it that you added an extra feature and your neat HTML structure :)
I think your implementation would benefit from adding a background color for the whole page, so that the notifications container stands out more. Additionally, you missed some hover states - bold texts should change color when hovered over.
When I implemented this project, I also made it more challenging by adding the ability to interact with each notification (or ´article´ in your case) - so that the user can read them one by one. I learned a lot from doing this, so you may want to give it a try too.
Hope it helps!
Marked as helpful
@nyrellcl
Posted
Hello @katjadanilova !
Oh yes! I’ll make sure to add that background color. Weird, the hover states were working when I submitted, maybe I have some styling errors, I’ll make sure to double check. Thanks for checking that!
Oh okay! Your feature sounds like another fun idea to implement, I’ve been thinking of what else I could do to expand this project a little bit more haha.
Thanks again!