scroll to top on route change

zurfyx · Aug 25, 2017 · 6cb0ce8 · 6cb0ce8
1 parent ede3154
commit 6cb0ce8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/src/app/core/sidenav/sidenav.component.html b/src/app/core/sidenav/sidenav.component.html
@@ -5,7 +5,7 @@
   </md-sidenav>
   <app-navbar></app-navbar>
   <div class="below-navbar">
-    <router-outlet></router-outlet>
+    <router-outlet (deactivate)="onDeactivate()"></router-outlet>
   </div>
 </md-sidenav-container>
 
@@ -14,6 +14,6 @@
     <app-sidenav-links></app-sidenav-links>
   </md-sidenav>
   <div class="below-navbar">
-    <router-outlet></router-outlet>
+    <router-outlet (deactivate)="onDeactivate()"></router-outlet>
   </div>
 </md-sidenav-container>
diff --git a/src/app/core/sidenav/sidenav.component.ts b/src/app/core/sidenav/sidenav.component.ts
@@ -14,4 +14,8 @@ export class SidenavComponent {
   constructor(sidenavService: SidenavService) {
     this.isMobileNavbarOpen = sidenavService.isMobileNavbarOpen;
   }
+
+  onDeactivate() {
+    window.scrollTo(0, 0);
+  }
 }
diff --git a/src/app/story/detail/photo/story-detail-photo.component.ts b/src/app/story/detail/photo/story-detail-photo.component.ts
@@ -180,7 +180,6 @@ export class StoryDetailPhotoComponent extends StoryDetailEditComponent {
 
   /**
    * Delete a photo that has been previously stored.
-   * TODO.
    */
   deleteStoryPhoto(photo: { url: string, title?: string }) {
     this.setPending();