check first to see if we already have a container generated by Mojito…

… and don't render a template unless it's model has changes
nonopolarity · Jan 4, 2013 · 83966df · 83966df
1 parent 532b858
commit 83966df
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/lib/app/autoload/mojito-yaf-client.client.js b/lib/app/autoload/mojito-yaf-client.client.js
@@ -453,10 +453,12 @@ YUI.add('mojito-yaf-client', function(Y, NAME) {
                     model.after('change', this.render, this);
                 }
 
-                container = Y.Node.create('<div id="' + params.id +
-                                             '" class="mojit"></div>');
-                if (attachPoint = this.getDOMAttachPoint()) {
-                    attachPoint.append(container);
+                if (!(container = Y.one('#' + params.id))) {
+                    container = Y.Node.create('<div id="' + params.id +
+                                                 '" class="mojit"></div>');
+                    if (attachPoint = this.getDOMAttachPoint()) {
+                        attachPoint.append(container);
+                    }
                 }
 
                 container.getDOMNode()._mojit = params.mojit;
@@ -518,15 +520,17 @@ YUI.add('mojito-yaf-client', function(Y, NAME) {
 
                 if (!(model = this.get('model'))) {
                     html = this.get('templateEngine').render(
-                                                    template,
-                                                    null);
+                                                template,
+                                                null);
                 } else {
-                    html = this.get('templateEngine').render(
-                                                    template,
-                                                    this.get('model').toJSON());
+                    if (Object.keys(model.changed).length > 0) {
+                        html = this.get('templateEngine').render(
+                                                template,
+                                                this.get('model').toJSON());
+                    }
                 }
 
-                container.append(html);
+                container.setHTML(html);
             }
         }, {
             ATTRS: {