fix bug in which hiddenNodes was reference on gview.get('hidden_node'). So any changes on hiddenNodes was modifying the original array too, preventing us to come back to the original state of the view.
authorrougeronj
Thu, 18 Jun 2015 16:25:21 +0200
changeset 486 f1ff0529ba41
parent 485 a95d4a631bcf
child 487 48be7ebb3187
fix bug in which hiddenNodes was reference on gview.get('hidden_node'). So any changes on hiddenNodes was modifying the original array too, preventing us to come back to the original state of the view.
client/js/renderer/scene.js
--- a/client/js/renderer/scene.js	Tue Jun 16 11:38:38 2015 +0200
+++ b/client/js/renderer/scene.js	Thu Jun 18 16:25:21 2015 +0200
@@ -244,15 +244,16 @@
         this.$.find(".Rk-ZoomSetSaved").click( function() {
             var view = _this.renkan.project.get("views").last();
             if(view){
+                _this.showNodes(false);
                 _this.setScale(view.get("zoom_level"), new paper.Point(view.get("offset")));
-                _this.hiddenNodes = view.get("hidden_nodes") || [];
+                _this.hiddenNodes = (view.get("hidden_nodes") || []).concat();
                 _this.hideNodes();
             }
         });
         this.$.find(".Rk-ShowHiddenNodes").mouseenter( function() {
             _this.showNodes(true);
             _this.$.find(".Rk-ShowHiddenNodes").mouseleave( function() {
-                _this.hideNodes(false);
+                _this.hideNodes();
             });
         });
         this.$.find(".Rk-ShowHiddenNodes").click( function() {
@@ -897,7 +898,9 @@
         },
         showNodes: function(ghost){
             var _this = this;
+            var i = 0;
             this.hiddenNodes.forEach(function(_id){
+                i++;
                 _this.getRepresentationByModel(_this.renkan.project.get("nodes").get(_id)).show(ghost);
             });
             if (!ghost){