Merge pull request #2739 from plotly/polar-polygon-grids

Polar polygon grids

Author: etpinard

Diff for: src/plots/polar/layout_attributes.js

@@ -122,11 +122,6 @@ var radialAxisAttrs = {
     // span: {},
     // hole: 1
 
-    // maybe should add a boolean to enable square grid lines
-    // and square axis lines
-    // (most common in radar-like charts)
-    // e.g. squareline/squaregrid or showline/showgrid: 'square' (on-top of true)
-
     editType: 'calc'
 };
 
@@ -272,6 +267,22 @@ module.exports = {
     radialaxis: radialAxisAttrs,
     angularaxis: angularAxisAttrs,
 
+    gridshape: {
+        valType: 'enumerated',
+        values: ['circular', 'linear'],
+        dflt: 'circular',
+        role: 'style',
+        editType: 'plot',
+        description: [
+            'Determines if the radial axis grid lines and angular axis line are drawn',
+            'as *circular* sectors or as *linear* (polygon) sectors.',
+            'Has an effect only when the angular axis has `type` *category*.',
+            'Note that `radialaxis.angle` is snapped to the angle of the closest',
+            'vertex when `gridshape` is *circular*',
+            '(so that radial axis scale is the same as the data scale).'
+        ].join(' ')
+    },
+
     // TODO maybe?
     // annotations:
 

Diff for: src/plots/polar/layout_defaults.js

@@ -177,6 +177,10 @@ function handleDefaults(contIn, contOut, coerce, opts) {
 
         axOut._input = axIn;
     }
+
+    if(contOut.angularaxis.type === 'category') {
+        coerce('gridshape');
+    }
 }
 
 function handleAxisTypeDefaults(axIn, axOut, coerce, subplotData, dataAttr) {