From 223dd99a2096f43fd0d559c634bd2c7c0145562b Mon Sep 17 00:00:00 2001
From: Rene Saarsoo <nene@triin.net>
Date: Tue, 23 Aug 2011 14:10:41 -0700
Subject: [PATCH] Fix saving video/examples tabs.

Previously refreshing the page while example tab open, would close all other tabs.
Now first loading all the tabs and only after that performing history navigation.
---
 template/app/History.js         | 19 ++++++++++++++++++-
 template/app/controller/Tabs.js |  6 +++++-
 2 files changed, 23 insertions(+), 2 deletions(-)

diff --git a/template/app/History.js b/template/app/History.js
index 8e247669..83965030 100644
--- a/template/app/History.js
+++ b/template/app/History.js
@@ -9,11 +9,28 @@ Ext.define("Docs.History", {
      */
     init: function() {
         Ext.util.History.init(function() {
-            this.navigate(Ext.util.History.getToken());
+            this.historyLoaded = true;
+            this.initialNavigate();
         }, this);
+
         Ext.util.History.on("change", this.navigate, this);
     },
 
+    /**
+     * Called from Tabs controller to notify that initial tabs loading had been done.
+     */
+    notifyTabsLoaded: function() {
+        this.tabsLoaded = true;
+        this.initialNavigate();
+    },
+
+    // Invoke initial navigation only after both tabs and history are loaded
+    initialNavigate: function() {
+        if (this.tabsLoaded && this.historyLoaded) {
+            this.navigate(Ext.util.History.getToken());
+        }
+    },
+
     // Parses current URL and navigates to the page
     navigate: function(token) {
         var url = this.parseToken(token);
diff --git a/template/app/controller/Tabs.js b/template/app/controller/Tabs.js
index 5a27a45e..9c0a1251 100644
--- a/template/app/controller/Tabs.js
+++ b/template/app/controller/Tabs.js
@@ -3,7 +3,10 @@
  */
 Ext.define('Docs.controller.Tabs', {
     extend: 'Ext.app.Controller',
-    requires: ['Docs.Settings'],
+    requires: [
+        'Docs.History',
+        'Docs.Settings'
+    ],
 
     refs: [
         {
@@ -93,6 +96,7 @@ Ext.define('Docs.controller.Tabs', {
                 this.addTabFromTree(url, {});
             }, this);
         }
+        Docs.History.notifyTabsLoaded();
     },
 
     /**
-- 
GitLab