UX: Do not scroll-top for aborted transitions (#22744)

We only want to scroll to the top for successful transitions. If a transition is aborted (e.g. when clicking a chat link when chat is in drawer mode) then we should maintain the existing scroll location.

app/assets/javascripts/discourse/app/services/route-scroll-manager.js

@@ -38,6 +38,10 @@ export default class RouteScrollManager extends Service {
 
   @bind
   routeDidChange(transition) {
+    if (transition.isAborted) {
+      return;
+    }
+
     const newUuid = this.router.location.getState?.().uuid;
 
     if (newUuid === this.uuid) {