Summary

Fixes a bug where pressing Enter on a notification would stop working after viewing any notification. The first Enter would work, but subsequent presses (on the same or different notifications) would have no effect.

Problem

The NotificationsView key handler used a switch statement where the PR/Issue subject checks came before the Enter key check:

switch {
case m.notificationView.GetSubjectPR() != nil:
    // ALL keys routed here when PR subject is set
    // Enter gets absorbed, never reaches loadNotificationContent()
    
case key.Matches(msg, keys.NotificationKeys.View):  // Enter
    // Never reached if GetSubjectPR() != nil
}

Once a notification was viewed and the subject set, Enter would be absorbed by the PR/Issue handler instead of triggering loadNotificationContent().

Solution

Move the Enter key handling before the subject checks:

switch {
case key.Matches(msg, keys.NotificationKeys.View):  // Enter
    cmds = append(cmds, m.loadNotificationContent())

case m.notificationView.GetSubjectPR() != nil:
    // PR-specific keys handled here
}

Also clears the cached subject in syncSidebar() when navigating to a different notification, ensuring key dispatch routes correctly.