#121: source button to play nice with hashes - cont'd

getredash · Mar 11, 2014 · 56672a8 · 56672a8
1 parent b5e5fb2
commit 56672a8
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/rd_ui/app/scripts/controllers/query_view.js b/rd_ui/app/scripts/controllers/query_view.js
@@ -13,19 +13,23 @@
 
     $scope.isSourceVisible = route.locals.viewSource;
 
-    $scope.sourceHref = $scope.isSourceVisible ?
-      $location.url().replace('src', '') : $location.path() + '/src#' + $location.hash();
-
     $scope.queryExecuting = false;
 
     $scope.newVisualization = undefined;
 
+    updateSourceHref();
+
     $window.onbeforeunload = function() {
       if ($scope.canEdit && $scope.dirty) {
         return leavingPageText;
       }
     }
 
+    function updateSourceHref() {
+      $scope.sourceHref = $scope.isSourceVisible ?
+        $location.url().replace('src', '') : $location.path() + '/src#' + $location.hash();
+    };
+
     Mousetrap.bindGlobal("meta+s", function(e) {
       e.preventDefault();
 
@@ -56,6 +60,7 @@
       return $location.hash()
     }, function(hash) {
       $scope.selectedTab = hash || DEFAULT_TAB;
+      updateSourceHref();
     });
 
     $scope.lockButton = function(lock) {