Merge pull request #729 from plotly/icons

use new cone/streamtube icons

Author: nicolaskruchten

package.json

@@ -14,7 +14,7 @@
     "draft-js-utils": "^1.2.0",
     "fast-isnumeric": "^1.1.1",
     "immutability-helper": "^2.7.1",
-    "plotly-icons": "latest",
+    "plotly-icons": "1.2.2",
     "plotly.js": "1.40.1",
     "prop-types": "^15.5.10",
     "raf": "^3.4.0",

src/lib/index.js

@@ -51,16 +51,8 @@ function renderTraceIcon(trace, prefix = 'Plot') {
     return null;
   }
   const gl = 'gl';
-
-  let tempTrace = trace;
-  if (tempTrace === 'cone') {
-    tempTrace = 'scatter3d';
-  } else if (tempTrace === 'streamtube') {
-    tempTrace = 'line3d';
-  }
-
   const componentName = `${prefix}${pascalCase(
-    tempTrace.endsWith(gl) ? tempTrace.slice(0, -gl.length) : tempTrace
+    trace.endsWith(gl) ? trace.slice(0, -gl.length) : trace
   )}Icon`;
 
   return PlotlyIcons[componentName] ? PlotlyIcons[componentName] : PlotlyIcons.PlotLineIcon;

src/lib/traceTypes.js

@@ -110,13 +110,11 @@ export const traceTypes = _ => [
   {
     value: 'cone',
     label: _('Cone'),
-    icon: 'scatter3d',
     category: chartCategory(_).THREE_D,
   },
   {
     value: 'streamtube',
     label: _('Streamtube'),
-    icon: 'line3d',
     category: chartCategory(_).THREE_D,
   },
   {