Bug 427568 - Navigation Menu transition should not happen on page load

--Signed-off-by: Elijah El-Haddad <elijahe@ca.ibm.com>
elijahe · Feb 6, 2014 · 22eec8e · 22eec8e
1 parent 5248d30
commit 22eec8e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/bundles/org.eclipse.orion.client.ui/web/css/layout.css b/bundles/org.eclipse.orion.client.ui/web/css/layout.css
@@ -125,11 +125,14 @@ hard-coded numbers elsewhere.
 	/* bottom: 36px; $FooterHeight + footer border (1px) */    
     bottom: 0;
     left: 0;
-    transition: left 0.3s ease; /* if this is changed SideMenu.js->TRANSITION_DURATION_MS needs to be changed as well */
     right: 0;
 	/*margin-bottom: 29px;*/
 }
 
+.content-fixedHeight-animation {
+    transition: left 0.3s ease; /* if this is changed SideMenu.js->TRANSITION_DURATION_MS needs to be changed as well */
+}
+
 .content-fixedHeight-maximized {
 	top: 0;
 	bottom: 0;

diff --git a/bundles/org.eclipse.orion.client.ui/web/orion/webui/SideMenu.js b/bundles/org.eclipse.orion.client.ui/web/orion/webui/SideMenu.js
@@ -101,6 +101,13 @@ define([
 			var sideMenuNavigation = this.getDisplayState();
 
 			var newState;
+
+			// add animation if necessary
+			var pageContent = lib.node( "pageContent" ); //$NON-NLS-0$
+			if (pageContent) {
+				pageContent.classList.add("content-fixedHeight-animation"); //$NON-NLS-0$
+			}
+
 			if( sideMenuNavigation === this.OPEN_STATE ){
 				newState = this.CLOSED_STATE;
 			} else {