[$200] In app notification updates

[jmgasper]

Currently

One of our testers found that there are times when they receive multiple notifications it can take up the whole screen.

Expected behaviour

Remove Content Preview.

Ex: for the bottom notification it should only read:
Forum beta Test - INVITE ONLY: holysyid comments on Sample Dataset for this challenge.

Max notifications

Only allow max 5 notifications at once. If there are more than 5 notifications, fade out and fade in next set.

Marvel

The MarvelApp has been updated to address: https://marvelapp.com/prototype/c06bg41/screen/77911797

[jmgasper]

Challenge https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been created for this ticket.

This is an automated message for ghostar via Topcoder X

[jmgasper]

Challenge https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been assigned to obog.

This is an automated message for ghostar via Topcoder X

[atelomycterus]

@jmgasper Let me know if you want to replace commented on with comments on as displayed in MarvelApp. The phrases are stored in DB. There is no UI to change them. The list of all phrases in the Phrases section

Please apply PRs:
topcoder-platform/forums-theme#34
topcoder-platform/forums-plugins#80
#516

Thanks!

Notifications

Only allow max 5 notifications at once. If there are more than 5 notifications, fade out and fade in next set.

No changes. Max 5 notifications are displayed by default. If user is in the middle of a visit only get very recent notifications (DateUpdated > -5 mins ago).

Phrases

Testing

Please clear browser cache before testing.

Desktop:

Mobile:

[jmgasper]

@atelomycterus - I think we're fine to leave as commented on for now, thanks!