fix(ui): move noDataTimeLine creation to later stage

move noDataTimeLine creation to later stage to accomidate IE9 not generate tabs fast enough. Closes #9608
RAMP-PCAR · May 12, 2015 · fcded2e · fcded2e
1 parent dcba250
commit fcded2e
Showing 1 changed file with 20 additions and 7 deletions.
diff --git a/src/js/RAMP/Modules/gui.js b/src/js/RAMP/Modules/gui.js
@@ -968,13 +968,6 @@ define([
                 }
             ];
 
-            // timeline that hides Data tab
-            noDataTimeLine
-                .fromTo(panelDiv.find(".wb-tabs > ul li:first"), transitionDuration, { width: "50%" }, { width: "100%", className: "+=h5", ease: "easeOutCirc" }, 0)
-                .to(panelDiv.find(".wb-tabs > ul li:first"), transitionDuration, { lineHeight: '20px' }, 0)
-                .fromTo(panelDiv.find(".wb-tabs > ul li:last"), transitionDuration, { width: "50%" }, { width: "0%", display: "none", ease: "easeOutCirc" }, 0)
-            ;
-
             /**
             * Fires an event when the layout of the page changes.
             *
@@ -1146,6 +1139,22 @@ define([
                 }
             }
 
+            /**
+             * Setup noDataTimeLine for data tab animation
+             * 
+             * @method setupNoDataTimeLine
+             * @private
+             */
+            function setupNoDataTimeLine() {
+
+                // timeline that hides Data tab
+                noDataTimeLine
+                    .fromTo(panelDiv.find(".wb-tabs > ul li:first"), transitionDuration, { width: "50%" }, { width: "100%", className: "+=h5", ease: "easeOutCirc" }, 0)
+                    .to(panelDiv.find(".wb-tabs > ul li:first"), transitionDuration, { lineHeight: '20px' }, 0)
+                    .fromTo(panelDiv.find(".wb-tabs > ul li:last"), transitionDuration, { width: "50%" }, { width: "0%", display: "none", ease: "easeOutCirc" }, 0)
+                ;
+            }
+
             return {
                 /**
                 * Initializes layout controller.
@@ -1200,6 +1209,10 @@ define([
                     }
 
                     adjustHeight();
+
+                    // setup noDataTimeLine here to allow IE9 to have enough time to 
+                    // create the tab
+                    setupNoDataTimeLine();
                 },
 
                 /**