Hi,

when clicking on the "Mark all as read" icon within the notifications popover the page in the background jumps to the top.
This is not the case with the "Mark all as read" icon in the messages popover...

Steps to reproduce

1. Login as a user
2. Scroll down the page a little bit
3. Click on the "notifications" icon in the navbar
4. In the popover header click on the "Mark all as read" icon
5. => See that the page in the background jumps to the top
6. Try the same steps with the messages
7. => See that the page in the background stays at it's position the user scrolled to

Thoughts
I've had a look at the code and compared the HTML structure and also the Code behind it.
The methods notification_popover_controller.js and message_popover_controller.js differ in the method "markAllAsRead"

• https://github.com/moodle/moodle/blob/master/message/output/popup/amd/src/notification_popover_controller.js#L311
• https://github.com/moodle/moodle/blob/master/message/output/popup/amd/src/message_popover_controller.js#L244).

I've tried to complete the method from the notification_popover_conroller.js with the code from the message_popover_controller.js but it did not solve this problem.
So I really don't know why both buttons behave differently.... But it would be great if both buttons would behave equally.

Best, Kathrin