apluslms / a-plus

A+ frontend portal - A+ LMS documentation:
https://apluslms.github.io/
Other
65 stars 72 forks source link

Reintroduce animation for new notifications but make it only run a set number of cycles #1288

Closed oseppala closed 9 months ago

oseppala commented 10 months ago

Course notifications are critical for course communication. They notify the student of feedback provided by the instructors of the course.

In commit f6afe4e animation was removed from the new notification badge. The original problem was that a notification could not be cleared, not that the animation needed to be removed.

A suggested fix is to revert the changes in commit f6afe4e and adding a cycle count to the CSS animation. The animation below only runs twice.

animation: bounce 10s 2 ease-in-out;

A separate issue about clearing notifications on past courses is in #1289