New Notification UX

[SteveAmor]

This is about the UX when new notifications are received. Currently, when Notifications are enabled and you get a new notification, the motor vibrates and the new notification is displayed with a timeoutLine. After 7 seconds (timeoutLength), the display returns to the previous screen.
This discussion is about the touch events during timeoutLenght which is managed at:

InfiniTime/src/displayapp/screens/Notifications.cpp

Line 151 in 0f9f606

bool Notifications::OnTouchEvent(Pinetime::Applications::TouchEvents event) {

It appears to react to 3 touch events:

• Tap - timeoutLine is deleted before the screen times out and goes to sleep, remaining on the notification when next woken up. When woken up, if there are previous notifications, you cannot swipe down to see them, you can only swipe right to dismiss or swipe up to go to previous screen.
• Swipe Right - dismisses the notification and returns to the previous screen.
• Swipe Up - takes you to the previous screen.

I can't see where the code allows Swipe Up during timeoutLenght but, aside from that, the UX doesn't seem right, having Tap. I'd propose:

• Swipe Down - takes you to a previous notification if there was one. And you would then have to Swipe Up to go to the next notification.
• Swipe Right - dismiss notification and show previous notification if it existed, else display 0 notifications.
• Swipe Up - takes you to the previous screen.

All three would delete the timeoutLine. All three would be consistent with swiping notifications.
At the very least, I think the Tap event should be removed, leaving only Swipe Right and Swipe Up.