From 6c994c4b90023cecf4fd0caafb404b859fe28f54 Mon Sep 17 00:00:00 2001
From: hwc487
Date: Wed, 6 Jun 2012 16:34:41 -0700
Subject: material cleanup and rearchitecture

---
 js/io/system/ninjalibrary.json                     |   2 +-
 js/lib/drawing/world.js                            |   2 +-
 js/lib/geom/circle.js                              |  22 ++-
 js/lib/geom/geom-obj.js                            |  21 +-
 js/lib/geom/rectangle.js                           |  23 ++-
 js/lib/rdge/materials/bump-metal-material.js       |   2 -
 js/lib/rdge/materials/cloud-material.js            |  17 --
 js/lib/rdge/materials/deform-material.js           |  16 --
 js/lib/rdge/materials/flag-material.js             |  16 --
 js/lib/rdge/materials/flat-material.js             |  18 +-
 js/lib/rdge/materials/fly-material.js              |   1 +
 js/lib/rdge/materials/material.js                  | 219 +++++++++++++++++----
 js/lib/rdge/materials/pulse-material.js            | 189 +++---------------
 js/lib/rdge/materials/taper-material.js            | 102 +++++-----
 js/lib/rdge/materials/water-material.js            |  45 +++--
 js/models/materials-model.js                       |  89 +++++----
 .../materials-popup.reel/materials-popup.js        |  16 ++
 17 files changed, 406 insertions(+), 394 deletions(-)

(limited to 'js')

diff --git a/js/io/system/ninjalibrary.json b/js/io/system/ninjalibrary.json
index 5c1eb875..5038bd21 100644
--- a/js/io/system/ninjalibrary.json
+++ b/js/io/system/ninjalibrary.json
@@ -1,6 +1,6 @@
 {
 	"libraries": [
 		{"name": "Montage", "path": "/node_modules/descriptor.json", "version": "0.10.0.0"},
-		{"name": "RDGE", "path": "/assets/descriptor.json", "version": "0.5.7.0"}
+		{"name": "RDGE", "path": "/assets/descriptor.json", "version": "0.5.8.2"}
 	]
 }
\ No newline at end of file
diff --git a/js/lib/drawing/world.js b/js/lib/drawing/world.js
index 228c0d94..8068284e 100755
--- a/js/lib/drawing/world.js
+++ b/js/lib/drawing/world.js
@@ -484,11 +484,11 @@ World.prototype.updateObject = function (obj) {
 	for (var i = 1; i < nPrims; i++)
 	{
 		// get the next primitive
-		var prim = prims[i];
 		childTrNode = RDGE.createTransformNode("objNode_" + this._nodeCounter++);
 		ctrTrNode.insertAsChild(childTrNode);
 
 		// attach the instanced box goe
+		var prim = prims[i];
 		childTrNode.attachMeshNode(this.renderer.id + "_prim_" + this._nodeCounter++, prim);
 		childTrNode.attachMaterial(materialNodes[i]);
 	}
diff --git a/js/lib/geom/circle.js b/js/lib/geom/circle.js
index b2709ce4..6627b4b7 100755
--- a/js/lib/geom/circle.js
+++ b/js/lib/geom/circle.js
@@ -57,16 +57,18 @@ exports.Circle = Object.create(GeomObj, {
             this.m_world = world;
 
             if(strokeMaterial) {
-                this._strokeMaterial = strokeMaterial;
+                this._strokeMaterial = strokeMaterial.dup();
             } else {
-                this._strokeMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() );
+                this._strokeMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() ).dup();
             }
+			if (strokeColor && this._strokeMaterial.hasProperty( "color" ))  this._strokeMaterial.setProperty( "color",  this._strokeColor );
 
             if(fillMaterial) {
-                this._fillMaterial = fillMaterial;
+                this._fillMaterial = fillMaterial.dup();
             } else {
-                this._fillMaterial = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
+                this._fillMaterial = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
             }
+			if (fillColor && this._fillMaterial.hasProperty( "color" ))  this._fillMaterial.setProperty( "color",  this._fillColor );
         }
     },
 
@@ -702,19 +704,23 @@ exports.Circle = Object.create(GeomObj, {
             var strokeMaterialName	= jObj.strokeMat;
             var fillMaterialName	= jObj.fillMat;
 
-            var strokeMat = MaterialsModel.getMaterial( strokeMaterialName );
+            var strokeMat = MaterialsModel.getMaterial( strokeMaterialName ).dup();
             if (!strokeMat) {
                 console.log( "object material not found in library: " + strokeMaterialName );
-                strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
+                strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
             }
             this._strokeMaterial = strokeMat;
+			if (this._strokeMaterial.hasProperty( 'color' ))
+				this._strokeMaterial.setProperty( 'color', this._strokeColor );
 
-            var fillMat = MaterialsModel.getMaterial( fillMaterialName );
+            var fillMat = MaterialsModel.getMaterial( fillMaterialName ).dup();
             if (!fillMat) {
                 console.log( "object material not found in library: " + fillMaterialName );
-                fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
+                fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
             }
             this._fillMaterial = fillMat;
+			if (this._fillMaterial.hasProperty( 'color' ))
+				this._fillMaterial.setProperty( 'color', this._fillColor );
 
             this.importMaterialsJSON( jObj.materials );
         }
diff --git a/js/lib/geom/geom-obj.js b/js/lib/geom/geom-obj.js
index 417c8731..3a7a5619 100755
--- a/js/lib/geom/geom-obj.js
+++ b/js/lib/geom/geom-obj.js
@@ -281,9 +281,11 @@ exports.GeomObj = Object.create(Object.prototype, {
             this._materialArray.push(strokeMaterial);
             this._materialTypeArray.push("stroke");
 
-            if (this._strokeColor) {
-                this.setStrokeColor(this._strokeColor);
-            }
+			// don't set the value here.  The material editor may set a color directly
+			// to the material without setting this value in the obj.  The following
+			// lines of code will clobber the value in the material
+            //if (this._strokeColor)
+            //    this.setStrokeColor(this._strokeColor);
 
 			this._strokeMaterial = strokeMaterial;
 
@@ -306,12 +308,14 @@ exports.GeomObj = Object.create(Object.prototype, {
 
             this._materialArray.push(fillMaterial);
             this._materialTypeArray.push("fill");
+			
+			// don't set the value here.  The material editor may set a color directly
+			// to the material without setting this value in the obj.  The following
+			// lines of code will clobber the value in the material
+            //if (this._fillColor)
+            //     this.setFillColor(this._fillColor);
 
-            if (this._fillColor) {
-                this.setFillColor(this._fillColor);
-            }
-
-        this._fillMaterial = fillMaterial;
+			this._fillMaterial = fillMaterial;
 
             return fillMaterial;
         }
@@ -389,6 +393,7 @@ exports.GeomObj = Object.create(Object.prototype, {
                     case "radialBlur":
                     case "pulse":
 					case "twistVert":
+					case "taper":
                         mat = MaterialsModel.getMaterialByShader(shaderName);
                         if (mat)  mat = mat.dup();
                         break;
diff --git a/js/lib/geom/rectangle.js b/js/lib/geom/rectangle.js
index 42d51e74..81a8556d 100755
--- a/js/lib/geom/rectangle.js
+++ b/js/lib/geom/rectangle.js
@@ -73,16 +73,19 @@ exports.Rectangle = Object.create(GeomObj, {
             this._materialSpecular = [0.4, 0.4, 0.4,  1.0];
 
             if(strokeMaterial) {
-                this._strokeMaterial = strokeMaterial;
+                this._strokeMaterial = strokeMaterial.dup();
             } else {
-                this._strokeMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() );
+                this._strokeMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() ).dup();
             }
+			if (strokeColor && this._strokeMaterial.hasProperty( "color" ))  this._strokeMaterial.setProperty( "color",  this._strokeColor );
+
 
             if(fillMaterial) {
-                this._fillMaterial = fillMaterial;
+                this._fillMaterial = fillMaterial.dup();
             } else {
-                this._fillMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() );
+                this._fillMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() ).dup();
             }
+			if (fillColor && this._fillMaterial.hasProperty( "color" ))  this._fillMaterial.setProperty( "color",  this._fillColor );
         }
     },
 
@@ -289,19 +292,23 @@ exports.Rectangle = Object.create(GeomObj, {
             var strokeMaterialName	= jObj.strokeMat;
             var fillMaterialName	= jObj.fillMat;
 
-            var strokeMat = MaterialsModel.getMaterial( strokeMaterialName );
+            var strokeMat = MaterialsModel.getMaterial( strokeMaterialName ).dup();
             if (!strokeMat) {
                 console.log( "object material not found in library: " + strokeMaterialName );
-                strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
+                strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
             }
             this._strokeMaterial = strokeMat;
+			if (this._strokeMaterial.hasProperty( 'color' ))
+				this._strokeMaterial.setProperty( 'color', this._strokeColor );
 
-            var fillMat = MaterialsModel.getMaterial( fillMaterialName );
+            var fillMat = MaterialsModel.getMaterial( fillMaterialName ).dup();
             if (!fillMat) {
                 console.log( "object material not found in library: " + fillMaterialName );
-                fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
+                fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
             }
             this._fillMaterial = fillMat;
+			if (this._fillMaterial.hasProperty( 'color' ))
+				this._fillMaterial.setProperty( 'color', this._fillColor );
 
             this.importMaterialsJSON( jObj.materials );
         }
diff --git a/js/lib/rdge/materials/bump-metal-material.js b/js/lib/rdge/materials/bump-metal-material.js
index 4d19c9c1..30624f7b 100755
--- a/js/lib/rdge/materials/bump-metal-material.js
+++ b/js/lib/rdge/materials/bump-metal-material.js
@@ -108,8 +108,6 @@ var BumpMetalMaterial = function BumpMetalMaterial() {
     ///////////////////////////////////////////////////////////////////////
     // Methods
     ///////////////////////////////////////////////////////////////////////
-	// duplcate method requirde
-	this.dup = function()	{  return new BumpMetalMaterial();	};
 
 	this.init = function( world )
 	{
diff --git a/js/lib/rdge/materials/cloud-material.js b/js/lib/rdge/materials/cloud-material.js
index 2c7fced2..2153b857 100644
--- a/js/lib/rdge/materials/cloud-material.js
+++ b/js/lib/rdge/materials/cloud-material.js
@@ -106,23 +106,6 @@ var CloudMaterial = function CloudMaterial()
     ///////////////////////////////////////////////////////////////////////
     // duplicate method required
     /**************************************************************/
-	this.dup = function (world) {
-        // save the world
-        if (world) this.setWorld(world);
-
-        // allocate a new uber material
-        var newMat = new CloudMaterial();
-
-        // copy over the current values;
-        var propNames = [], propValues = [], propTypes = [], propLabels = [];
-        this.getAllProperties(propNames, propValues, propTypes, propLabels);
-        var n = propNames.length;
-        for (var i = 0; i < n; i++) {
-            newMat.setProperty(propNames[i], propValues[i]);
-        }
-
-        return newMat;
-    };
 
     this.init = function (world) {
 		var GLWorld = require("js/lib/drawing/world").World,
diff --git a/js/lib/rdge/materials/deform-material.js b/js/lib/rdge/materials/deform-material.js
index c7f9cadc..70742c77 100644
--- a/js/lib/rdge/materials/deform-material.js
+++ b/js/lib/rdge/materials/deform-material.js
@@ -37,22 +37,6 @@ var DeformMaterial = function DeformMaterial() {
     // Methods
     ///////////////////////////////////////////////////////////////////////
     // duplcate method requirde
-    this.dup = function (world)
-	{
-        // get the current values;
-        var propNames = [], propValues = [], propTypes = [], propLabels = [];
-        this.getAllProperties(propNames, propValues, propTypes, propLabels);
-        
-        // allocate a new material
-        var newMat = new DeformMaterial();
-
-		// copy over the current values;
-        var n = propNames.length;
-        for (var i = 0; i < n; i++)
-            newMat.setProperty(propNames[i], propValues[i]);
-
-        return newMat;
-    };
 
     this.init = function (world) {
         // save the world
diff --git a/js/lib/rdge/materials/flag-material.js b/js/lib/rdge/materials/flag-material.js
index d1788fb8..aaf36ebf 100644
--- a/js/lib/rdge/materials/flag-material.js
+++ b/js/lib/rdge/materials/flag-material.js
@@ -51,22 +51,6 @@ var FlagMaterial = function FlagMaterial() {
     // Methods
     ///////////////////////////////////////////////////////////////////////
 	// duplcate method requirde
-	this.dup = function( world )
-	{
-		// get the current values;
-        var propNames = [], propValues = [], propTypes = [], propLabels = [];
-        this.getAllProperties(propNames, propValues, propTypes, propLabels);
-        
-        // allocate a new uber material
-        var newMat = new FlagMaterial();
-
-		// copy over the current values;
-        var n = propNames.length;
-        for (var i = 0; i < n; i++)
-            newMat.setProperty(propNames[i], propValues[i]);
-
-        return newMat;
-	};
 
 	this.setProperty = function( prop, value )
 	{
diff --git a/js/lib/rdge/materials/flat-material.js b/js/lib/rdge/materials/flat-material.js
index 40da9305..e5498b65 100755
--- a/js/lib/rdge/materials/flat-material.js
+++ b/js/lib/rdge/materials/flat-material.js
@@ -34,7 +34,22 @@ var FlatMaterial = function FlatMaterial() {
     // Methods
     ///////////////////////////////////////////////////////////////////////
     // duplcate method requirde
-    this.dup = function () { return new FlatMaterial(); };
+	this.dup = function( world )
+	{
+        // get the current values;
+        var propNames = [], propValues = [], propTypes = [], propLabels = [];
+        this.getAllProperties(propNames, propValues, propTypes, propLabels);
+        
+        // allocate a new material
+        var newMat = new FlatMaterial();
+
+		// copy over the current values;
+        var n = propNames.length;
+        for (var i = 0; i < n; i++)
+            newMat.setProperty(propNames[i], propValues[i]);
+
+        return newMat;
+	};
 
     this.init = function (world) {
         // save the world
@@ -72,6 +87,7 @@ var FlatMaterial = function FlatMaterial() {
         // make sure we have legitimate input
         if (this.validateProperty(prop, value)) {
             this._propValues[prop] = value;
+			if (prop === 'color')  this._color = value.slice();
             if (this._shader && this._shader.colorMe) {
                 this._shader.colorMe[prop].set(value);
             }
diff --git a/js/lib/rdge/materials/fly-material.js b/js/lib/rdge/materials/fly-material.js
index ca38cc83..5e14f3af 100644
--- a/js/lib/rdge/materials/fly-material.js
+++ b/js/lib/rdge/materials/fly-material.js
@@ -106,6 +106,7 @@ var flyMaterialDef =
 				{
 					'u_tex0': { 'type' : 'tex2d' },
 					'u_time' : { 'type' : 'float' },
+					'u_speed'  : { 'type' : 'float' },
 					'u_resolution'  :   { 'type' : 'vec2' },
 				},
 
diff --git a/js/lib/rdge/materials/material.js b/js/lib/rdge/materials/material.js
index 856513e2..d2586b58 100755
--- a/js/lib/rdge/materials/material.js
+++ b/js/lib/rdge/materials/material.js
@@ -4,6 +4,9 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
 (c) Copyright 2011 Motorola Mobility, Inc.  All Rights Reserved.
 </copyright> */
 
+var Texture = require("js/lib/rdge/texture").Texture;
+
+
 ///////////////////////////////////////////////////////////////////////
 // Class GLMaterial
 //      GL representation of a material.
@@ -15,19 +18,16 @@ var Material = function GLMaterial( world ) {
 	this._name = "GLMaterial";
 	this._shaderName = "undefined";
 
+	this._time = 0.0;
+	this._dTime = 0.01;
+
 	// keep a reference to the owning GLWorld
 	this._world = null;
     if(world) {
         this._world = world;
     }
 
-	this._shininess = 60;
-		
-	this._ambient  = [0.0, 0.0, 0.0,  1.0];
-	this._diffuse  = [0.0, 0.0, 0.0,  1.0];
-	this._specular = [0.0, 0.0, 0.0,  1.0];
-
-	this._texture = null;
+	this._glTextures = [];		// indexed by uniform name
 
 	// vertex deformation variables
 	this._hasVertexDeformation = false;
@@ -46,13 +46,6 @@ var Material = function GLMaterial( world ) {
     ///////////////////////////////////////////////////////////////////////
     // Property Accessors
     ///////////////////////////////////////////////////////////////////////
-	this.getShininess = function() {
-        return this._shininess;
-    };
-
-	this.setShininess = function(s) {
-        this._shininess = s;
-    };
 
 	this.setName = function(n) {
         this._name = n;
@@ -78,30 +71,6 @@ var Material = function GLMaterial( world ) {
         return this._world;
     };
 
-	this.setAmbient	= function(r, g, b, a)	{
-        this._ambient = [r, g, b, a];
-    };
-
-	this.getAmbient = function() {
-        return [this._ambient[0], this._ambient[1], this._ambient[2], this._ambient[3]];
-    };
-
-	this.setDiffuse	= function(r, g, b, a)	{
-        this._diffuse = [r, g, b, a];
-    };
-
-	this.getDiffuse	= function() {
-        return [this._diffuse[0], this._diffuse[1], this._diffuse[2], this._diffuse[3]];
-    };
-
-	this.setSpecular = function(r, g, b, a)	{
-        this._specular = [r, g, b, a];
-    };
-
-	this.getSpecular = function() {
-        return [this._specular[0], this._specular[1], this._specular[2], this._specular[3]];
-    };
-
 	this.getShader = function() {
         return this._shader;
     };
@@ -116,6 +85,10 @@ var Material = function GLMaterial( world ) {
         return false;
     };
 
+	this.getTechniqueName	= function() {
+		return 'default'
+	};
+
 	// the vertex shader can apply deformations requiring refinement in
 	// certain areas.
 	this.hasVertexDeformation = function()	{
@@ -166,6 +139,43 @@ var Material = function GLMaterial( world ) {
 		}
 	};
 
+	this.hasProperty = function( prop )
+	{
+		var propNames = [],  dummy = [];
+		this.getAllProperties( propNames, dummy, dummy, dummy )
+		for (var i=0;  i<propNames.length;  i++)
+		{
+			if (prop === propNames[i])  return true;
+		}
+	};
+
+	this.getPropertyType = function( prop )
+	{
+		var n = this.getPropertyCount();
+		for (var i=0;  i<n;  i++)
+		{
+			if (prop === this._propNames[i])  return this._propTypes[i];
+		}
+	};
+
+    this.dup = function ()
+	{
+        // get the current values;
+        var propNames = [], propValues = [], propTypes = [], propLabels = [];
+        this.getAllProperties(propNames, propValues, propTypes, propLabels);
+        
+        // allocate a new material
+		var MaterialLibrary = require("js/models/materials-model").MaterialsModel;
+        var newMat = MaterialLibrary.createMaterialByShaderName( this.getShaderName() );
+
+		// copy over the current values;
+        var n = propNames.length;
+        for (var i = 0; i < n; i++)
+            newMat.setProperty(propNames[i], propValues[i]);
+
+        return newMat;
+    };
+
 	this.validateProperty = function( prop, value ) {
 		var rtnVal = false;
 		try
@@ -216,16 +226,139 @@ var Material = function GLMaterial( world ) {
 
 		return rtnVal;
 	};
+
+	this.setProperty = function( prop,  value )
+	{
+		var ok = this.validateProperty( prop, value );
+		if (!ok && (prop != 'color')) {
+			console.log( "invalid property in Material:" + prop + " : " + value );
+			return;
+		}
+
+		// get the technique if the shader is instantiated
+		var technique;
+		var material = this._materialNode;
+		if (material)  technique = material.shaderProgram[this.getTechniqueName()];
+
+		switch (this.getPropertyType(prop))
+		{
+			case "angle":
+			case "float":
+				this._propValues[prop] = value;
+				if (technique)  technique[prop].set( [value] );
+				break;
+
+			case "file":
+				this._propValues[prop] = value.slice();
+				if (technique)
+				{
+					var glTex = new Texture( this.getWorld(),  value );
+					this._glTextures[prop] = glTex;
+					glTex.render();
+					var tex = glTex.getTexture();
+					if (tex)  technique[prop].set( tex );
+				}
+				break;
+
+			case "color":
+			case "vector2d":
+			case "vector3d":
+				this._propValues[prop] = value.slice();
+				if (technique)  technique[prop].set( value );
+				break;
+		}
+	};
+
+	this.setShaderValues = function()
+	{
+		var material = this._materialNode;
+		if (material)
+		{
+			var technique = material.shaderProgram[this.getTechniqueName()];
+			if (technique)
+			{
+				var n = this.getPropertyCount();
+				for (var i=0;  i<n;  i++)
+				{
+					var prop = this._propNames[i],
+						value = this._propValues[prop];
+
+					switch (this._propTypes[i])
+					{
+						case "angle":
+						case "float":
+							technique[prop].set( [value] );
+							break;
+
+						case "file":
+							var glTex = this._glTextures[prop];
+							if (glTex)
+							{
+								var tex = glTex.getTexture();
+								if (tex)  technique[prop].set( tex );
+							}
+							else
+								this.setProperty( prop, value );
+							break;
+
+						case "color":
+						case "vector2d":
+						case "vector3d":
+							technique[prop].set( value );
+							break;
+					}
+				}
+			}
+		}
+	};
+
+	// JSON export
+	this.exportJSON = function()
+	{
+		var jObj =
+		{
+			'material'		: this.getShaderName(),
+			'name'			: this.getName(),
+			'dTime'         : this._dTime
+		};
+
+		var n = this.getPropertyCount();
+		for (var i=0;  i<n;  i++)
+		{
+			var	prop  = this._propNames[i],
+				value = this._propValues[prop];
+
+			jObj[prop] = value;
+		}
+
+		return jObj;
+	};
+	
+    this.importJSON = function (jObj) {
+        if (this.getShaderName() != jObj.material) throw new Error("ill-formed material");
+        this.setName(jObj.name);
+
+        try
+		{
+			for (var prop in jObj)
+			{
+				if ((prop != 'material') && (prop != 'name'))
+				{
+					var value = jObj[prop];
+					this.setProperty( prop, value );
+				}
+			}
+        }
+        catch (e) {
+            throw new Error("could not import material: " + jObj);
+        }
+    };
+
     ///////////////////////////////////////////////////////////////////////
     
     ///////////////////////////////////////////////////////////////////////
     // Methods
     ///////////////////////////////////////////////////////////////////////
-	// duplcate method required by sub class
-	this.dup = function() {
-		throw new Error( "Material.dup() must be overridden by subclass" );
-	};
-
 	this.init = function( world ) {
 		throw new Error( "Material.init() must be overridden by subclass" );
 	};
diff --git a/js/lib/rdge/materials/pulse-material.js b/js/lib/rdge/materials/pulse-material.js
index 374a5eae..7c00b836 100644
--- a/js/lib/rdge/materials/pulse-material.js
+++ b/js/lib/rdge/materials/pulse-material.js
@@ -30,69 +30,30 @@ var PulseMaterial = function PulseMaterial()
 	this._time = 0.0;
 	this._dTime = 0.01;
 
-	this._xScale = 0.5;
-	this._yScale = 0.4;
-	this._speed  = 1.0;
-
 	///////////////////////////////////////////////////////////////////////
 	// Property Accessors
 	///////////////////////////////////////////////////////////////////////
-	this.getName		= function()	{ return this._name;			};
-	this.getShaderName	= function()	{  return this._shaderName;		};
-
-	this.getTextureMap			= function()		{  return this._propValues[this._propNames[0]] ? this._propValues[this._propNames[0]].slice() : null	};
-	this.setTextureMap			= function(m)		{  this._propValues[this._propNames[0]] = m ? m.slice(0) : null;  this.updateTexture();  	};
-
-	this.isAnimated			= function()			{  return true;					};
+	this.isAnimated			= function()			{  return true;		};
+	this.getShaderDef		= function()			{  return pulseMaterialDef;	}
 
 	///////////////////////////////////////////////////////////////////////
 	// Material Property Accessors
 	///////////////////////////////////////////////////////////////////////
-	this._propNames			= ["texmap",		"xscale",		"yscale",		"speed" ];
+
+	var u_tex0_index	= 0,
+		u_xScale_index	= 1,
+		u_yScale_index	= 2,
+		u_speed_index	= 3;
+
+	this._propNames			= ["u_tex0",		"u_xscale",		"u_yscale",		"u_speed" ];
 	this._propLabels		= ["Texture map",	"X Range",		"Y Range",		"Speed" ];
 	this._propTypes			= ["file",			"float",		"float",		"float"];
 	this._propValues		= [];
 
-	this._propValues[ this._propNames[0] ] = this._texMap.slice(0);
-	this._propValues[ this._propNames[1] ] = this._xScale;
-	this._propValues[ this._propNames[2] ] = this._yScale;
-	this._propValues[ this._propNames[3] ] = this._speed;
-
-	this.setProperty = function( prop, value ) {
-		// make sure we have legitimate imput
-		var ok = this.validateProperty( prop, value );
-		if (!ok && (prop != 'color')) {
-			console.log( "invalid property in Material:" + prop + " : " + value );
-		}
-
-		switch (prop)
-		{
-			case "texmap":
-				this.setTextureMap(value);
-				break;
-
-			case "speed":
-				this._speed = value;
-				this._propValues[ this._propNames[3] ] = this._speed;
-				this.updateParameters();
-				break;
-
-			case "xscale":
-				this._xScale = value;
-				this._propValues[ this._propNames[1] ] = this._xScale;
-				this.updateParameters();
-				break;
-
-			case "yscale":
-				this._yScale = value;
-				this._propValues[ this._propNames[2] ] = this._yScale;
-				this.updateParameters();
-				break;
-
-			case "color":
-				break;
-		}
-	};
+	this._propValues[ this._propNames[  u_tex0_index] ] = this._texMap.slice(0);
+	this._propValues[ this._propNames[u_xScale_index] ] = 0.5;
+	this._propValues[ this._propNames[u_yScale_index] ] = 0.4;
+	this._propValues[ this._propNames[ u_speed_index] ] = 1.0;
 	///////////////////////////////////////////////////////////////////////
 
 
@@ -100,26 +61,9 @@ var PulseMaterial = function PulseMaterial()
 	// Methods
 	///////////////////////////////////////////////////////////////////////
 	// duplicate method required
-	this.dup = function( world ) {
-		// save the world
-		if (world)  this.setWorld( world );
-
-		// get the current values;
-		var propNames = [], propValues = [], propTypes = [], propLabels = [];
-		this.getAllProperties(propNames, propValues, propTypes, propLabels);
-		
-		// allocate a new material
-		var newMat = new PulseMaterial();
-
-		// copy over the current values;
-		var n = propNames.length;
-		for (var i = 0; i < n; i++)
-			newMat.setProperty(propNames[i], propValues[i]);
-
-		return newMat;
-	};
 
-	this.init = function( world ) {
+	this.init = function( world )
+	{
 		// save the world
 		if (world)  this.setWorld( world );
 
@@ -140,67 +84,17 @@ var PulseMaterial = function PulseMaterial()
 		if (this._shader && this._shader['default']) {
 			this._shader['default'].u_time.set( [this._time] );
 		}
-		this.updateParameters();
 
 		// set up the texture
-		var texMapName = this._propValues[this._propNames[0]];
-		this._glTex = new Texture( world, texMapName );
+		//var texMapName = this._propValues[this._propNames[0]];
+		//this.setProperty( "u_tex0",  texMapName );
 
 		// set the shader values in the shader
-		this.updateTexture();
+		this.setShaderValues();
 		this.setResolution( [world.getViewportWidth(),world.getViewportHeight()] );
 		this.update( 0 );
 	};
 
-	this.updateParameters = function()
-	{
-		this._propValues[ this._propNames[1] ] = this._xScale;
-		this._propValues[ this._propNames[2] ] = this._yScale;
-		this._propValues[ this._propNames[3] ] = this._speed;
-
-		var material = this._materialNode;
-		if (material)
-		{
-			var technique = material.shaderProgram['default'];
-			var renderer = RDGE.globals.engine.getContext().renderer;
-			if (renderer && technique)
-			{
-
-				if (this._shader && this._shader['default']) {
-					this._shader['default'].u_speed.set( [this._speed] );
-					this._shader['default'].u_xscale.set( [this._xScale] );
-					this._shader['default'].u_yscale.set( [this._yScale] );
-				}
-			}
-		}
-	}
-
-	this.updateTexture = function() {
-		
-		var texMapName = this._propValues[this._propNames[0]];
-		this._glTex = new Texture( this.getWorld(), texMapName );
-
-		var material = this._materialNode;
-		if (material) {
-			var technique = material.shaderProgram['default'];
-			var renderer = RDGE.globals.engine.getContext().renderer;
-			if (renderer && technique) {
-				var wrap = 'REPEAT',  mips = true;
-				var tex;
-				if (this._glTex)
-				{
-					if (this._glTex.isAnimated())
-						this._glTex.render();
-					tex = this._glTex.getTexture();
-				}
-				
-				if (tex) {
-					technique.u_tex0.set( tex );
-				}
-			}
-		}
-	};
-
 	this.update = function( time )
 	{
 		var material = this._materialNode;
@@ -210,12 +104,13 @@ var PulseMaterial = function PulseMaterial()
 			var renderer = RDGE.globals.engine.getContext().renderer;
 			if (renderer && technique)
 			{
-				if (this._glTex)
+				var glTex = this._glTextures["u_tex0"];
+				if (glTex)
 				{
 					//this.updateTexture();
-					if (this._glTex.isAnimated())
-						this._glTex.render();
-					tex = this._glTex.getTexture();
+					if (glTex.isAnimated())
+						glTex.render();
+					tex = glTex.getTexture();
 					if (tex)
 						technique.u_tex0.set( tex );
 				}
@@ -239,44 +134,6 @@ var PulseMaterial = function PulseMaterial()
 		}
 	};
 
-	// JSON export
-	this.exportJSON = function()
-	{
-		var jObj =
-		{
-			'material'		: this.getShaderName(),
-			'name'			: this.getName(),
-			'texture'		: this._propValues[this._propNames[0]],
-			'dTime'         : this._dTime,
-			'speed'			: this._speed,
-			'xScale'		: this._xScale,
-			'yScale'		: this._yScale
-		};
-
-		return jObj;
-	};
-
-	this.importJSON = function( jObj ) {
-		if (this.getShaderName() != jObj.material)  throw new Error( "ill-formed material" );
-		this.setName(  jObj.name );
-
-		try {
-			this._propValues[this._propNames[0]] = jObj.texture;
-			this._texMap = jObj.texture;
-			if (jObj.dTime) {
-				this._dTime = jObj.dTime;
-			}
-
-			this._speed = jObj.speed;
-			this._xScale = jObj.xScale;
-			this._yScale = jObj.yScale;
-			this.updateParameters();
-		}
-		catch (e)
-		{
-			throw new Error( "could not import material: " + jObj );
-		}
-	};
 };
 
 ///////////////////////////////////////////////////////////////////////////////////////
diff --git a/js/lib/rdge/materials/taper-material.js b/js/lib/rdge/materials/taper-material.js
index 7938336b..c491fe61 100644
--- a/js/lib/rdge/materials/taper-material.js
+++ b/js/lib/rdge/materials/taper-material.js
@@ -23,14 +23,11 @@ var TaperMaterial = function TaperMaterial()
     this._name = "TaperMaterial";
     this._shaderName = "taper";
 
-    this._color = [1, 0, 0, 1];
-
     this._deltaTime = 0.0;
 
     ///////////////////////////////////////////////////////////////////////
     // Property Accessors
     ///////////////////////////////////////////////////////////////////////
-    this.getColor = function () { return this._color; };
     this.getShaderName = function () { return this._shaderName; };
 
     this.isAnimated = function () { return true; };
@@ -42,7 +39,6 @@ var TaperMaterial = function TaperMaterial()
     // Methods
     ///////////////////////////////////////////////////////////////////////
     // duplcate method requirde
-    this.dup = function () { return new TaperMaterial(); };
 
     this.init = function (world) {
         this.setWorld(world);
@@ -52,9 +48,6 @@ var TaperMaterial = function TaperMaterial()
         this._shader.def = taperShaderDef;
         this._shader.init();
 
-        // set the defaults
-        this._shader.colorMe.color.set(this.getColor());
-
         // set up the material node
         this._materialNode = RDGE.createMaterialNode("taperMaterial" + "_" + world.generateUniqueNodeID());
         this._materialNode.setShader(this._shader);
@@ -67,41 +60,46 @@ var TaperMaterial = function TaperMaterial()
     ///////////////////////////////////////////////////////////////////////
     // Material Property Accessors
     ///////////////////////////////////////////////////////////////////////
-    this._propNames = ["color", "u_limit1", "u_limit2", "u_limit3", "u_minVal", "u_maxVal", "u_center", "u_taperAmount"];
-    this._propLabels = ["Color", "Minimum Parameter Value", "Center Paramater Value", "Maximum Parameter Value", "Minimum Data Bounds", "Maximum Data Bounds", "Center", "Taper Amount"];
-    this._propTypes = ["color", "float", "float", "float", "float", "float", "float", "float"];
+    this._propNames = [ "u_limit1", "u_limit2", "u_limit3", "u_minVal", "u_maxVal", "u_center", "u_taperAmount",  "u_speed" ];
+    this._propLabels = [ "Minimum Parameter Value", "Center Paramater Value", "Maximum Parameter Value", "Minimum Data Bounds", "Maximum Data Bounds", "Center", "Taper Amount", "Speed" ];
+    this._propTypes = [ "float", "float", "float", "float", "float", "float", "float", "float" ];
     this._propValues = [];
 
     // initialize the property values
-    this._propValues[this._propNames[0]] = this._color.slice();
-    this._propValues[this._propNames[1]] = 0.25;
-    this._propValues[this._propNames[2]] = 0.50;
-    this._propValues[this._propNames[3]] = 0.75;
-    this._propValues[this._propNames[4]] = -1;
-    this._propValues[this._propNames[5]] = 1;
-    this._propValues[this._propNames[6]] = 0.0;
-    this._propValues[this._propNames[7]] = 0.9;
-
-    this.setProperty = function (prop, value) {
+    this._propValues[this._propNames[0]] = 0.25;
+    this._propValues[this._propNames[1]] = 0.50;
+    this._propValues[this._propNames[2]] = 0.75;
+    this._propValues[this._propNames[3]] = -1;
+    this._propValues[this._propNames[4]] = 1;
+    this._propValues[this._propNames[5]] = 0.0;
+    this._propValues[this._propNames[6]] = 0.9;
+    this._propValues[this._propNames[7]] = 1.0;
+
+    this.setProperty = function (prop, value)
+	{
         // make sure we have legitimate input
-        if (this.validateProperty(prop, value)) {
-            switch (prop) {
-                case "color": this._propValues[prop] = value.slice(); break;
-                default: this._propValues[prop] = value; break;
-            }
+        if (this.validateProperty(prop, value))
+               this._propValues[prop] = value;
 
-            this.updateShaderValues();
-        }
+        this.updateShaderValues();
     };
-    ///////////////////////////////////////////////////////////////////////
+
     this.exportJSON = function () {
         var jObj =
 		{
 		    'material': this.getShaderName(),
-		    'name': this.getName(),
-		    'color': this._propValues["color"]
+		    'name': this.getName()
 		};
 
+		var n = this._propNames.length;
+		for (var i=0;  i<n;  i++)
+		{
+			var prop = this._propNames[i],
+				val  = this._propValues[prop];
+
+			jObj[prop] = val;
+		}
+
         return jObj;
     };
 
@@ -109,9 +107,16 @@ var TaperMaterial = function TaperMaterial()
         if (this.getShaderName() != jObj.material) throw new Error("ill-formed material");
         this.setName(jObj.name);
 
-        try {
-            var color = jObj.color;
-            this.setProperty("color", color);
+        try
+		{
+			for ( var prop in jObj)
+			{
+				if ((prop != 'material') && (prop != 'name'))
+				{
+					var value = jObj[prop];
+					this.setProperty( prop, value );
+				}
+			}
         }
         catch (e) {
             throw new Error("could not import material: " + jObj);
@@ -119,23 +124,23 @@ var TaperMaterial = function TaperMaterial()
     };
 
     this.update = function (time) {
-        //var speed = 0.01;
-        //time *= speed;
-        this._deltaTime += 0.01;
+        var speed = this._propValues["u_speed"];
+        this._deltaTime += 0.01 * speed;
 
         if (this._shader && this._shader.colorMe) {
-            var t3 = this._propValues[this._propNames[3]] - this._deltaTime;
+            var t3 = this._propValues["u_limit3"] - this._deltaTime;
             if (t3 < 0) {
-                this._deltaTime = this._propValues[this._propNames[1]] - 1.0;
-                t3 = this._propValues[this._propNames[3]] - this._deltaTime;
+                this._deltaTime = this._propValues["u_limit1"] - 1.0;
+                t3 = this._propValues["u_limit3"] - this._deltaTime;
             }
-            var t1 = this._propValues[this._propNames[1]] - this._deltaTime,
-				t2 = this._propValues[this._propNames[2]] - this._deltaTime;
 
+            var t1 = this._propValues["u_limit1"] - this._deltaTime,
+				t2 = this._propValues["u_limit2"] - this._deltaTime;
 
-            this._shader.colorMe[this._propNames[1]].set([t1]);
-            this._shader.colorMe[this._propNames[2]].set([t2]);
-            this._shader.colorMe[this._propNames[3]].set([t3]);
+
+            this._shader.colorMe["u_limit1"].set([t1]);
+            this._shader.colorMe["u_limit2"].set([t2]);
+            this._shader.colorMe["u_limit3"].set([t3]);
         }
     };
 
@@ -145,10 +150,9 @@ var TaperMaterial = function TaperMaterial()
             for (var i = 0; i < nProps; i++) {
                 var propName = this._propNames[i];
                 var propValue = this._propValues[propName];
-                switch (propName) {
-                    case "color": this._shader.colorMe[propName].set(propValue); break;
-                    default: this._shader.colorMe[propName].set([propValue]); break;
-                }
+
+                if (this._shader.colorMe[propName])
+					this._shader.colorMe[propName].set([propValue]);
             }
         }
     };
@@ -179,8 +183,6 @@ taperShaderDef = {
 			// attributes
 			'params':
 				 {
-				     'color': { 'type': 'vec4' },
-
 				     'u_limit1': { 'type': 'float' },
 				     'u_limit2': { 'type': 'float' },
 				     'u_limit3': { 'type': 'float' },
diff --git a/js/lib/rdge/materials/water-material.js b/js/lib/rdge/materials/water-material.js
index 1f0aa46e..8dbb3233 100644
--- a/js/lib/rdge/materials/water-material.js
+++ b/js/lib/rdge/materials/water-material.js
@@ -50,22 +50,6 @@ var WaterMaterial = function WaterMaterial() {
     ///////////////////////////////////////////////////////////////////////
     // Methods
     ///////////////////////////////////////////////////////////////////////
-    // duplcate method requirde
-    this.dup = function (world) {
-        // get the current values;
-        var propNames = [], propValues = [], propTypes = [], propLabels = [];
-        this.getAllProperties(propNames, propValues, propTypes, propLabels);
-        
-        // allocate a new material
-        var newMat = new WaterMaterial();
-
-		// copy over the current values;
-        var n = propNames.length;
-        for (var i = 0; i < n; i++)
-            newMat.setProperty(propNames[i], propValues[i]);
-
-        return newMat;
-    };
 
 	this.setProperty = function( prop, value ) {
 		// make sure we have legitimate imput
@@ -142,6 +126,32 @@ var WaterMaterial = function WaterMaterial() {
 		this.updateTexture();
 	}
 
+	this.updateTexture = function() {
+		
+		var texMapName = this._propValues[this._propNames[0]];
+		this._glTex = new Texture( this.getWorld(), texMapName );
+
+		var material = this._materialNode;
+		if (material) {
+			var technique = material.shaderProgram['default'];
+			var renderer = RDGE.globals.engine.getContext().renderer;
+			if (renderer && technique) {
+				var wrap = 'REPEAT',  mips = true;
+				var tex;
+				if (this._glTex)
+				{
+					if (this._glTex.isAnimated())
+						this._glTex.render();
+					tex = this._glTex.getTexture();
+				}
+				
+				if (tex) {
+					technique.u_tex0.set( tex );
+				}
+			}
+		}
+	};
+
 	this.setEmboss = function( value )
 	{
 		this._emboss = value;
@@ -251,6 +261,7 @@ var waterMaterialDef =
 	}
 };
 
+/*
 var ParisMaterial = function ParisMaterial() {
     // initialize the inherited members
     this.inheritedFrom = WaterMaterial;
@@ -349,7 +360,7 @@ var parisMaterialDef =
 		]
 	}
 };
-
+*/
 
 WaterMaterial.prototype = new PulseMaterial();
 
diff --git a/js/models/materials-model.js b/js/models/materials-model.js
index 20ef317a..c3b111f4 100755
--- a/js/models/materials-model.js
+++ b/js/models/materials-model.js
@@ -27,7 +27,7 @@ var FlagMaterial = require("js/lib/rdge/materials/flag-material").FlagMaterial;
 var SquareTunnelMaterial = require("js/lib/rdge/materials/square-tunnel-material").SquareTunnelMaterial;
 var FlyMaterial = require("js/lib/rdge/materials/fly-material").FlyMaterial;
 var WaterMaterial = require("js/lib/rdge/materials/water-material").WaterMaterial;
-var ParisMaterial = require("js/lib/rdge/materials/water-material").ParisMaterial;
+//var ParisMaterial = require("js/lib/rdge/materials/water-material").ParisMaterial;
 var ZInvertMaterial = require("js/lib/rdge/materials/z-invert-material").ZInvertMaterial;
 var DeformMaterial = require("js/lib/rdge/materials/deform-material").DeformMaterial;
 var StarMaterial = require("js/lib/rdge/materials/star-material").StarMaterial;
@@ -60,7 +60,7 @@ exports.MaterialsModel = Montage.create(Component, {
 			this.addMaterial(new MandelMaterial());
 			this.addMaterial(new PlasmaMaterial());
 			this.addMaterial(new PulseMaterial());
-			this.addMaterial(new ParisMaterial());
+			//this.addMaterial(new ParisMaterial());
 			this.addMaterial(new RadialGradientMaterial());
 			this.addMaterial(new RadialBlurMaterial());
 			this.addMaterial(new RaidersMaterial());
@@ -210,6 +210,51 @@ exports.MaterialsModel = Montage.create(Component, {
 		}
 	},
 
+	createMaterialByShaderName:
+	{
+		value: function(shaderName)
+		{
+			var mat;
+			switch (shaderName)
+			{
+				case "flat":				mat = new FlatMaterial();				break;
+				case "linearGradient":		mat = new LinearGradientMaterial();		break;
+				case "radialGradient":		mat = new RadialGradientMaterial();		break;
+				case "bumpMetal":			mat = new BumpMetalMaterial();			break;
+				case "uber":				mat = new UberMaterial();				break;
+				case "cloud":				mat = new CloudMaterial();				break;
+
+				case "taper":				mat = new TaperMaterial();				break;
+				case "twistVert":			mat = new TwistVertMaterial();			break;
+				case "radialBlur":			mat = new RadialBlurMaterial();			break;
+				case "plasma":				mat = new PlasmaMaterial();				break;
+				case "pulse":				mat = new PulseMaterial();				break;
+				case "tunnel":				mat = new TunnelMaterial();				break;
+				case "reliefTunnel":		mat = new ReliefTunnelMaterial();		break;
+				case "squareTunnel":		mat = new SquareTunnelMaterial();		break;
+				case "flag":		        mat = new FlagMaterial();		        break;
+				case "fly":					mat = new FlyMaterial();				break;
+				case "water":				mat = new WaterMaterial();				break;
+				//case "paris":				mat = new ParisMaterial();				break;
+				case "raiders":		        mat = new RaidersMaterial();		    break;
+				case "zinvert":				mat = new ZInvertMaterial();			break;
+				case "deform":				mat = new DeformMaterial();				break;
+				case "star":				mat = new StarMaterial();				break;
+				case "twist":				mat = new TwistMaterial();				break;
+				case "julia":				mat = new JuliaMaterial();				break;
+				case "keleidoscope":		mat = new KeleidoscopeMaterial();		break;
+				case "mandel":				mat = new MandelMaterial();				break;
+
+
+				default:
+					console.log( "Unrecognized shader name: " + shaderName );
+					break;
+			}
+
+			return mat;
+		}
+	},
+
 	importMaterials: {
 		value: function( jObj )
         {
@@ -224,45 +269,9 @@ exports.MaterialsModel = Montage.create(Component, {
 			var nMats = matArray.length;
             for (var i=0;  i<nMats;  i++)
             {
-              var mat = null;
-              var jMatObj = matArray[i];
+                var jMatObj = matArray[i];
                 var type = jMatObj.material;
-                switch (type)
-                {
-					case "flat":				mat = new FlatMaterial();				break;
-					case "linearGradient":		mat = new LinearGradientMaterial();		break;
-					case "radialGradient":		mat = new RadialGradientMaterial();		break;
-					case "bumpMetal":			mat = new BumpMetalMaterial();			break;
-					case "uber":				mat = new UberMaterial();				break;
-					case "cloud":				mat = new CloudMaterial();				break;
-
-					case "taper":				mat = new TaperMaterial();				break;
-					case "twistVert":			mat = new TwistVertMaterial();			break;
-					case "radialBlur":			mat = new RadialBlurMaterial();			break;
-					case "plasma":				mat = new PlasmaMaterial();				break;
-					case "pulse":				mat = new PulseMaterial();				break;
-					case "tunnel":				mat = new TunnelMaterial();				break;
-					case "reliefTunnel":		mat = new ReliefTunnelMaterial();		break;
-					case "squareTunnel":		mat = new SquareTunnelMaterial();		break;
-					case "flag":		        mat = new FlagMaterial();		        break;
-					case "fly":					mat = new FlyMaterial();				break;
-					case "water":				mat = new WaterMaterial();				break;
-					case "paris":				mat = new ParisMaterial();				break;
-					case "raiders":		        mat = new RaidersMaterial();		    break;
-					case "zinvert":				mat = new ZInvertMaterial();			break;
-					case "deform":				mat = new DeformMaterial();				break;
-					case "star":				mat = new StarMaterial();				break;
-					case "twist":				mat = new TwistMaterial();				break;
-					case "julia":				mat = new JuliaMaterial();				break;
-					case "keleidoscope":		mat = new KeleidoscopeMaterial();		break;
-					case "mandel":				mat = new MandelMaterial();				break;
-
-
-					default:
-						throw new Error( "Unrecognized material type: " + type );
-						break;
-				}
-
+				var mat = this.createMaterialByShaderName( type );
 				if (mat) {
 					importStr = mat.importJSON( jMatObj );
 					this.addMaterial( mat );
diff --git a/js/panels/Materials/materials-popup.reel/materials-popup.js b/js/panels/Materials/materials-popup.reel/materials-popup.js
index 3ef2e4ef..5b9790b8 100755
--- a/js/panels/Materials/materials-popup.reel/materials-popup.js
+++ b/js/panels/Materials/materials-popup.reel/materials-popup.js
@@ -19,6 +19,7 @@ exports.MaterialsPopup = Montage.create(Component, {
     },
 
 	_useSelection: {  value: false,  enumerable: true },
+	_whichMaterial: { value: "fill", enumerable: true },
 
 	captureAction: {
 		value:function(event) {
@@ -140,6 +141,20 @@ exports.MaterialsPopup = Montage.create(Component, {
 				if (this._useSelection)
 				{
 					console.log( "apply to selection" );
+
+					var selection = this.application.ninja.selectedElements;
+					if (selection && (selection.length > 0))
+					{
+						var canvas = selection[0];
+						var obj;
+						if (canvas.elementModel && canvas.elementModel.shapeModel)  obj = canvas.elementModel.shapeModel.GLGeomObj;
+						if (obj)
+						{
+							var world = obj.getWorld();
+							if (world)
+								world.restartRenderLoop();
+						}
+					}
 				}
 			}
 		}
@@ -241,6 +256,7 @@ exports.MaterialsPopup = Montage.create(Component, {
 				{
 					var canvas = selection[0];
 					var obj;
+					this._whichMaterial = whichMaterial;
 					if (canvas.elementModel && canvas.elementModel.shapeModel)  obj = canvas.elementModel.shapeModel.GLGeomObj;
 					if (obj)
 						material = (whichMaterial === 'stroke') ? obj.getStrokeMaterial() : obj.getFillMaterial();
-- 
cgit v1.2.3