Skip to content
OverviewSolutions
17
Comments
4
P
Peter Ramirez
@peterramirez18

All comments

  • Happy Phiri•430
    @happy-phiriSubmitted almost 2 years ago

    Feedback would be appreciated on this. Its not perfect. There are one or two things am not entirely happy with and will need to work on but its one more step for me in the right direction. Just trying to get to grips with JavaScript.

    Notifications Page made with Vanilla JavaScript

    1
    Peter Ramirez•370
    @peterramirez18Posted almost 2 years ago

    Good Good Phiri!

    I'll add "All messages read" like you, that look better.

    • You forgot to add the red circle when the notification is unread.
    Marked as helpful
    0
  • Peter Ramirez•370
    @peterramirez18Posted almost 2 years ago

    Nice work!

    • Use tracking class in letters 80k 803k 1.4k
    1
  • Peter Ramirez•370
    @peterramirez18Posted almost 2 years ago

    Nice work Gideon!

    • Check the STEP 3, there's a bug when I click on the check/checked
    Marked as helpful
    1
  • Peter Ramirez•370
    @peterramirez18Posted about 3 years ago

    hi Erwin! good job! Add background-repeat: repeat-x" to the body.

    Marked as helpful
    0