Use charm container in sidebar

app/subapps/browser/views/sidebar.js

 'use strict';
 
 
 /**
  * Browser SubApp Sidebar View handler.
  *
  * @module juju.browser
  * @submodule views
  *
  */
@@ skipping 94 matching lines @@
 
       container.setHTML(tplNode);
 
       // By default we grab the editorial content from the api to use for
       // display.
       this.get('store').sidebarEditorial({
         'success': function(data) {
           var sliderCharms = this.get('store').resultsToCharmlist(
               data.result.slider);
           var sliderContainer = container.one('.bws-left .slider');
-          this.slider = this._generateSliderWidget(sliderCharms);
-          if (this.slider) {
-            this.slider.render(sliderContainer);
-          }
+          var sliderCharmContainer = new Y.juju.widgets.browser.CharmContainer({
+            name: 'Featured Charms',
+            cutoff: 1,
+            children: sliderCharms.map(function(charm) {
+              return charm.getAttrs(); })
+          });
+          sliderCharmContainer.render(sliderContainer);
 
           // Add in the charm tokens for the new as well.
           var newContainer = container.one('.bws-left .new');
           var newCharms = this.get('store').resultsToCharmlist(
               data.result['new']);
-          newCharms.map(function(charm) {
-            var node = Y.Node.create('<div>'),
-                widget = new Y.juju.widgets.browser.CharmToken(
-                charm.getAttrs());
-            widget.render(node);
-            newContainer.append(node);
+          var newCharmContainer = new Y.juju.widgets.browser.CharmContainer({
+            name: 'New Charms',
+            cutoff: 2,
+            children: newCharms.map(function(charm) {
+              return charm.getAttrs(); })
           });
+          newCharmContainer.render(newContainer);

[jeff.pihach, 2013/04/08 21:52:30]

I just want to make sure that these containers are cleaned up somewhere.

 
           // Add the charms to the cache for use in other views.
           // Start with a reset to empty any current cached models.
           this._cacheCharms.reset(newCharms);
           this._cacheCharms.add(sliderCharms);
         },
 
         'failure': function(data, request) {
           var message;
           if (data && data.type) {
@@ skipping 39 matching lines @@
       // Bind our view to the events from the search widget used for controls.
       this._bindSearchWidgetEvents();
     }
 
   }, {
     ATTRS: {}
   });
 
 }, '0.1.0', {
   requires: [
+    'browser-charm-container',
     'browser-charm-slider',
     'browser-charm-token',
     'browser-search-widget',
     'juju-charm-store',
     'juju-models',
     'juju-templates',
     'subapp-browser-charmview',
     'subapp-browser-mainview',
     'view'
   ]
 });