don't allow for node level color props

Author: bnjm

components/lib/createArComponent.js

@@ -69,23 +69,15 @@ export default (mountConfig, propTypes = {}, nonUpdateablePropKeys = []) => {
   // any custom props (material, shape, ...)
   const nonNodePropKeys = keys(propTypes);
 
-  const parseMaterials = props => {
-    const material = props.material || {};
-
-    if (props.color) {
-      material.diffuse = { ...material.diffuse, color: props.color };
-    }
-
-    return {
-      ...props,
-      ...(props.shadowColor
-        ? { shadowColor: processColor(props.shadowColor) }
-        : {}),
-      ...(props.material
-        ? { material: processMaterialPropertyContents(material) }
-        : {}),
-    };
-  };
+  const parseMaterials = props => ({
+    ...props,
+    ...(props.shadowColor
+      ? { shadowColor: processColor(props.shadowColor) }
+      : {}),
+    ...(props.material
+      ? { material: processMaterialPropertyContents(props.material) }
+      : {}),
+  });
 
   const getNonNodeProps = props => ({
     ...pick(props, nonNodePropKeys),

components/lib/parseMaterial.js

@@ -14,16 +14,17 @@ export function processMaterialPropertyContents(material) {
     Object.keys(material),
     materialPropertiesWithMaps,
   );
-  // legacy support for old diffuse.color
-  const color =
-    typeof material.diffuse === 'string' ? material.diffuse : undefined;
 
   return propsToUpdate.reduce(
     (prev, curr) => ({
       ...prev,
       [curr]: {
         ...prev[curr],
-        color: processColor(color || prev[curr].color),
+        color: processColor(
+          curr === 'diffuse' && typeof prev[curr] === 'string'
+            ? prev[curr]
+            : prev[curr].color,
+        ),
       },
     }),
     material,