Merge pull request #4794 from plotly/scattermapbox-marker-angle-dflt

Fixup default angle of markers on scattermapbox

Author: archmoj

src/traces/scattermapbox/attributes.js

@@ -89,11 +89,13 @@ module.exports = overrideAll({
         },
         angle: {
             valType: 'number',
-            dflt: 0,
+            dflt: 'auto',
             role: 'style',
             arrayOk: true,
             description: [
-                'Sets the marker rotation, in degrees clockwise.'
+                'Sets the marker orientation from true North, in degrees clockwise.',
+                'When using the *auto* default, no rotation would be applied',
+                'in perspective views which is different from using a zero angle.'
             ].join(' ')
         },
         allowoverlap: {

src/traces/scattermapbox/convert.js

@@ -104,7 +104,7 @@ module.exports = function convert(gd, calcTrace) {
                 'icon-size': trace.marker.size / 10
             });
 
-            if('angle' in trace.marker) {
+            if('angle' in trace.marker && trace.marker.angle !== 'auto') {
                 Lib.extendFlat(symbol.layout, {
                 // unfortunately cant use {angle} do to this issue:
                 // https://github.com/mapbox/mapbox-gl-js/issues/873
@@ -258,7 +258,7 @@ function makeSymbolGeoJSON(calcTrace, gd) {
         getFillFunc(symbol) :
         blankFillFunc;
 
-    var fillAngle = (angle) ?
+    var fillAngle = (angle !== 'auto') ?
         getFillFunc(angle, true) :
         blankFillFunc;