Use newer format for addPanel for nav items

bfederle · bfederle · commit a16f5a123789 · 2012-02-21T16:20:44.000-08:00
diff --git a/ui/scripts/ui/core.js b/ui/scripts/ui/core.js
@@ -53,7 +53,6 @@
    * @param args CloudStack3 configuration
    */
   var showSection = function(sectionID, args) {
-    var $panel;
     var $browser = $('#browser div.container');
     var $navItem = $('#navigation').find('li').filter(function() {
       return $(this).hasClass(sectionID);
@@ -66,24 +65,24 @@
 
     // Reset browser panels
     $browser.cloudBrowser('removeAllPanels');
-    $panel = $browser.cloudBrowser('addPanel', {
+    $browser.cloudBrowser('addPanel', {
       title: _l(data.title),
-      data: ''
-    });
-
-    // Hide breadcrumb if this is the home section
-    if (args.home === sectionID) {
-      $('#breadcrumbs').find('li:first, div.end:last').hide();
-    }
-
-    // Append specified widget to view
-    if (data.show)
-      $panel.append(data.show(data));
-    else if (data.treeView)
-      $panel.treeView(data, { context: args.context });
-    else
-      $panel.listView(data, { context: args.context });
+      data: '',
+      complete: function($panel) {
+        // Hide breadcrumb if this is the home section
+        if (args.home === sectionID) {
+          $('#breadcrumbs').find('li:first, div.end:last').hide();
+        }
 
+        // Append specified widget to view
+        if (data.show)
+          $panel.append(data.show(data));
+        else if (data.treeView)
+          $panel.treeView(data, { context: args.context });
+        else
+          $panel.listView(data, { context: args.context });      
+      }
+    });
 
     return $navItem;
   };
