plotly
diff --git a/‎src/traces/scatterpolar/calc.js‎
Lines changed: 2 additions & 3 deletions b/‎src/traces/scatterpolar/calc.js‎
Lines changed: 2 additions & 3 deletions
diff --git a/‎test/image/baselines/polar_categories.png‎
8.06 KB b/‎test/image/baselines/polar_categories.png‎
8.06 KB
diff --git a/‎test/image/baselines/polar_direction.png‎
7.21 KB b/‎test/image/baselines/polar_direction.png‎
7.21 KB
diff --git a/‎test/image/baselines/polar_scatter.png‎
-5.08 KB b/‎test/image/baselines/polar_scatter.png‎
-5.08 KB
diff --git a/‎test/image/baselines/polar_subplots.png‎
4.83 KB b/‎test/image/baselines/polar_subplots.png‎
4.83 KB
diff --git a/‎test/image/mocks/polar_sector.json‎
Lines changed: 80 additions & 0 deletions b/‎test/image/mocks/polar_sector.json‎
Lines changed: 80 additions & 0 deletions
@@ -47,15 +47,14 @@ module.exports = function calc(gd, trace) {
         }
     }
 
-    Axes.expand(radialAxis, rArray, {tozero: true});
-    calcMarkerSize(trace, len);
+    var ppad = calcMarkerSize(trace, len);
+    Axes.expand(radialAxis, rArray, {tozero: true, ppad: ppad});
 
     if(angularAxis.type !== 'linear') {
         angularAxis.autorange = true;
         Axes.expand(angularAxis, thetaArray);
     }
 
-    // TODO needs to bump auto ranges !!!
     calcColorscale(trace);
     arraysToCalcdata(cd, trace);
     calcSelection(cd, trace);
 
@@ -608,6 +608,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -635,6 +639,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -662,6 +670,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -689,6 +701,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -716,6 +732,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -743,6 +763,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -770,6 +794,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -797,6 +825,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -824,6 +856,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -851,6 +887,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -878,6 +918,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -905,6 +949,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -932,6 +980,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -959,6 +1011,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -986,6 +1042,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -1013,6 +1073,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -1040,6 +1104,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -1067,6 +1135,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -1094,6 +1166,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
@@ -1121,6 +1197,10 @@
         ]
       },
       "radialaxis": {
+        "range": [
+          0,
+          5
+        ],
         "tickfont": {
           "size": 8
         }
Original file line number	Diff line number	Diff line change
`@@ -47,15 +47,14 @@ module.exports = function calc(gd, trace) {`
`47`	`47`	`}`
`48`	`48`	`}`
`49`	`49`
`50`		`- Axes.expand(radialAxis, rArray, {tozero: true});`
`51`		`- calcMarkerSize(trace, len);`
	`50`	`+ var ppad = calcMarkerSize(trace, len);`
	`51`	`+ Axes.expand(radialAxis, rArray, {tozero: true, ppad: ppad});`
`52`	`52`
`53`	`53`	`if(angularAxis.type !== 'linear') {`
`54`	`54`	`angularAxis.autorange = true;`
`55`	`55`	`Axes.expand(angularAxis, thetaArray);`
`56`	`56`	`}`
`57`	`57`
`58`		`- // TODO needs to bump auto ranges !!!`
`59`	`58`	`calcColorscale(trace);`
`60`	`59`	`arraysToCalcdata(cd, trace);`
`61`	`60`	`calcSelection(cd, trace);`
Original file line number	Diff line number	Diff line change
`@@ -608,6 +608,10 @@`
`608`	`608`	`]`
`609`	`609`	`},`
`610`	`610`	`"radialaxis": {`
	`611`	`+ "range": [`
	`612`	`+ 0,`
	`613`	`+ 5`
	`614`	`+ ],`
`611`	`615`	`"tickfont": {`
`612`	`616`	`"size": 8`
`613`	`617`	`}`
`@@ -635,6 +639,10 @@`
`635`	`639`	`]`
`636`	`640`	`},`
`637`	`641`	`"radialaxis": {`
	`642`	`+ "range": [`
	`643`	`+ 0,`
	`644`	`+ 5`
	`645`	`+ ],`
`638`	`646`	`"tickfont": {`
`639`	`647`	`"size": 8`
`640`	`648`	`}`
`@@ -662,6 +670,10 @@`
`662`	`670`	`]`
`663`	`671`	`},`
`664`	`672`	`"radialaxis": {`
	`673`	`+ "range": [`
	`674`	`+ 0,`
	`675`	`+ 5`
	`676`	`+ ],`
`665`	`677`	`"tickfont": {`
`666`	`678`	`"size": 8`
`667`	`679`	`}`
`@@ -689,6 +701,10 @@`
`689`	`701`	`]`
`690`	`702`	`},`
`691`	`703`	`"radialaxis": {`
	`704`	`+ "range": [`
	`705`	`+ 0,`
	`706`	`+ 5`
	`707`	`+ ],`
`692`	`708`	`"tickfont": {`
`693`	`709`	`"size": 8`
`694`	`710`	`}`
`@@ -716,6 +732,10 @@`
`716`	`732`	`]`
`717`	`733`	`},`
`718`	`734`	`"radialaxis": {`
	`735`	`+ "range": [`
	`736`	`+ 0,`
	`737`	`+ 5`
	`738`	`+ ],`
`719`	`739`	`"tickfont": {`
`720`	`740`	`"size": 8`
`721`	`741`	`}`
`@@ -743,6 +763,10 @@`
`743`	`763`	`]`
`744`	`764`	`},`
`745`	`765`	`"radialaxis": {`
	`766`	`+ "range": [`
	`767`	`+ 0,`
	`768`	`+ 5`
	`769`	`+ ],`
`746`	`770`	`"tickfont": {`
`747`	`771`	`"size": 8`
`748`	`772`	`}`
`@@ -770,6 +794,10 @@`
`770`	`794`	`]`
`771`	`795`	`},`
`772`	`796`	`"radialaxis": {`
	`797`	`+ "range": [`
	`798`	`+ 0,`
	`799`	`+ 5`
	`800`	`+ ],`
`773`	`801`	`"tickfont": {`
`774`	`802`	`"size": 8`
`775`	`803`	`}`
`@@ -797,6 +825,10 @@`
`797`	`825`	`]`
`798`	`826`	`},`
`799`	`827`	`"radialaxis": {`
	`828`	`+ "range": [`
	`829`	`+ 0,`
	`830`	`+ 5`
	`831`	`+ ],`
`800`	`832`	`"tickfont": {`
`801`	`833`	`"size": 8`
`802`	`834`	`}`
`@@ -824,6 +856,10 @@`
`824`	`856`	`]`
`825`	`857`	`},`
`826`	`858`	`"radialaxis": {`
	`859`	`+ "range": [`
	`860`	`+ 0,`
	`861`	`+ 5`
	`862`	`+ ],`
`827`	`863`	`"tickfont": {`
`828`	`864`	`"size": 8`
`829`	`865`	`}`
`@@ -851,6 +887,10 @@`
`851`	`887`	`]`
`852`	`888`	`},`
`853`	`889`	`"radialaxis": {`
	`890`	`+ "range": [`
	`891`	`+ 0,`
	`892`	`+ 5`
	`893`	`+ ],`
`854`	`894`	`"tickfont": {`
`855`	`895`	`"size": 8`
`856`	`896`	`}`
`@@ -878,6 +918,10 @@`
`878`	`918`	`]`
`879`	`919`	`},`
`880`	`920`	`"radialaxis": {`
	`921`	`+ "range": [`
	`922`	`+ 0,`
	`923`	`+ 5`
	`924`	`+ ],`
`881`	`925`	`"tickfont": {`
`882`	`926`	`"size": 8`
`883`	`927`	`}`
`@@ -905,6 +949,10 @@`
`905`	`949`	`]`
`906`	`950`	`},`
`907`	`951`	`"radialaxis": {`
	`952`	`+ "range": [`
	`953`	`+ 0,`
	`954`	`+ 5`
	`955`	`+ ],`
`908`	`956`	`"tickfont": {`
`909`	`957`	`"size": 8`
`910`	`958`	`}`
`@@ -932,6 +980,10 @@`
`932`	`980`	`]`
`933`	`981`	`},`
`934`	`982`	`"radialaxis": {`
	`983`	`+ "range": [`
	`984`	`+ 0,`
	`985`	`+ 5`
	`986`	`+ ],`
`935`	`987`	`"tickfont": {`
`936`	`988`	`"size": 8`
`937`	`989`	`}`
`@@ -959,6 +1011,10 @@`
`959`	`1011`	`]`
`960`	`1012`	`},`
`961`	`1013`	`"radialaxis": {`
	`1014`	`+ "range": [`
	`1015`	`+ 0,`
	`1016`	`+ 5`
	`1017`	`+ ],`
`962`	`1018`	`"tickfont": {`
`963`	`1019`	`"size": 8`
`964`	`1020`	`}`
`@@ -986,6 +1042,10 @@`
`986`	`1042`	`]`
`987`	`1043`	`},`
`988`	`1044`	`"radialaxis": {`
	`1045`	`+ "range": [`
	`1046`	`+ 0,`
	`1047`	`+ 5`
	`1048`	`+ ],`
`989`	`1049`	`"tickfont": {`
`990`	`1050`	`"size": 8`
`991`	`1051`	`}`
`@@ -1013,6 +1073,10 @@`
`1013`	`1073`	`]`
`1014`	`1074`	`},`
`1015`	`1075`	`"radialaxis": {`
	`1076`	`+ "range": [`
	`1077`	`+ 0,`
	`1078`	`+ 5`
	`1079`	`+ ],`
`1016`	`1080`	`"tickfont": {`
`1017`	`1081`	`"size": 8`
`1018`	`1082`	`}`
`@@ -1040,6 +1104,10 @@`
`1040`	`1104`	`]`
`1041`	`1105`	`},`
`1042`	`1106`	`"radialaxis": {`
	`1107`	`+ "range": [`
	`1108`	`+ 0,`
	`1109`	`+ 5`
	`1110`	`+ ],`
`1043`	`1111`	`"tickfont": {`
`1044`	`1112`	`"size": 8`
`1045`	`1113`	`}`
`@@ -1067,6 +1135,10 @@`
`1067`	`1135`	`]`
`1068`	`1136`	`},`
`1069`	`1137`	`"radialaxis": {`
	`1138`	`+ "range": [`
	`1139`	`+ 0,`
	`1140`	`+ 5`
	`1141`	`+ ],`
`1070`	`1142`	`"tickfont": {`
`1071`	`1143`	`"size": 8`
`1072`	`1144`	`}`
`@@ -1094,6 +1166,10 @@`
`1094`	`1166`	`]`
`1095`	`1167`	`},`
`1096`	`1168`	`"radialaxis": {`
	`1169`	`+ "range": [`
	`1170`	`+ 0,`
	`1171`	`+ 5`
	`1172`	`+ ],`
`1097`	`1173`	`"tickfont": {`
`1098`	`1174`	`"size": 8`
`1099`	`1175`	`}`
`@@ -1121,6 +1197,10 @@`
`1121`	`1197`	`]`
`1122`	`1198`	`},`
`1123`	`1199`	`"radialaxis": {`
	`1200`	`+ "range": [`
	`1201`	`+ 0,`
	`1202`	`+ 5`
	`1203`	`+ ],`
`1124`	`1204`	`"tickfont": {`
`1125`	`1205`	`"size": 8`
`1126`	`1206`	`}`