Only allow creation of new axes for cartesian types for now

Author: VeraZab

src/components/containers/Section.js

@@ -4,6 +4,7 @@ import {
   containerConnectedContextTypes,
   localize,
   unpackPlotProps,
+  traceTypeToAxisType,
 } from '../../lib';
 
 class Section extends Component {
@@ -36,7 +37,13 @@ class Section extends Component {
       }
 
       if ((child.type.plotly_editor_traits || {}).is_axis_creator) {
-        if (this.context.data.length > 1) {
+        const {data, fullContainer} = this.context;
+
+        // for now, only allowing for cartesian chart types
+        if (
+          data.length > 1 &&
+          traceTypeToAxisType(data[fullContainer.index].type) === 'cartesian'
+        ) {
           this.sectionVisible = true;
           return cloneElement(child, {
             isVisible: true,

src/components/fields/AxesSelector.js

@@ -30,6 +30,7 @@ export default class AxesSelector extends Component {
       <Field {...this.props} center>
         {hasSecondaryAxis ? (
           <Dropdown
+            label={_('Select Axis')}
             options={axesOptions.map(option => {
               if (option.value !== 'allaxes') {
                 return {

src/components/fields/AxisCreator.js

@@ -23,10 +23,9 @@ export class UnconnectedAxisCreator extends Component {
       context.data.filter(d => d.type === props.container.type).length === 1;
 
     function getNewSubplot(axis, subplot) {
-      const ok = isFirstTraceOfType
+      return isFirstTraceOfType
         ? axis
         : axis + (context.fullLayout._subplots[subplot].length + 1);
-      return ok;
     }
 
     function getAxisControl(label, attr, subplot, update) {

src/lib/getAllAxes.js

@@ -55,5 +55,9 @@ export function traceTypeToAxisType(traceType) {
     return category;
   }
 
+  if (traceType === 'pie') {
+    return null;
+  }
+
   throw new Error(`Sorry, could not find ${traceType} in any category.`);
 }