Merge branch 'master' into image-test

Author: etpinard

.gitignore

@@ -3,7 +3,4 @@ node_modules
 build/*
 !build/README.md
 
-dist/*
-!dist/README.md
-
 npm-debug.log

devtools/test_dashboard/index.html

@@ -11,12 +11,15 @@
 
     <script type="text/javascript" src="../../dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
 
-    <script type="text/javascript" src="../../dist/plotly.js" charset="utf-8"></script>
-    <script>PLOTLYENV = { TOPOJSON_URL: '../../src/assets/topojson/' };</script>
+    <!-- use dev plotly.js build -->
+    <script type="text/javascript" src="../../build/plotly.js" charset="utf-8"></script>
+
+    <!-- use local topojson files -->
+    <script>Plotly.setPlotConfig({ topojsonURL: '../../dist/topojson/' });</script>
 
     <script type="text/javascript" src="../../build/test_dashboard-bundle.js"></script>
 
-    <!-- helper functions to manipulate graph div -->
+    <!-- helper functions to manipulate the graph div -->
     <script>
     var Tabs = {
         getGraph: function() {

dist/plotly-geo-assets.js

@@ -65,6 +65,7 @@
     "delaunay-triangulate": "^1.1.6",
     "es6-promise": "^3.0.2",
     "fast-isnumeric": "^1.1.1",
+    "fs-extra": "^0.26.2",
     "gl-error2d": "^1.0.0",
     "gl-error3d": "^1.0.0",
     "gl-line2d": "^1.2.1",
@@ -88,6 +89,7 @@
     "node-sass": "^3.4.1",
     "right-now": "^1.0.0",
     "robust-orientation": "^1.1.3",
+    "sane-topojson": "^1.2.0",
     "superscript-text": "^1.0.0",
     "through2": "^2.0.0",
     "tinycolor2": "1.1.2",

dist/plotly-with-meta.js

@@ -1,17 +1,3 @@
-exports.topojsons = {};
+var saneTopojson = require('sane-topojson');
 
-exports.topojsons['world_110m'] = require('./topojson/world_110m.json');
-exports.topojsons['africa_110m'] = require('./topojson/africa_110m.json');
-exports.topojsons['asia_110m'] = require('./topojson/asia_110m.json');
-exports.topojsons['europe_110m'] = require('./topojson/europe_110m.json');
-exports.topojsons['north-america_110m'] = require('./topojson/north-america_110m.json');
-exports.topojsons['south-america_110m'] = require('./topojson/south-america_110m.json');
-exports.topojsons['usa_110m'] = require('./topojson/usa_110m');
-
-exports.topojsons['world_50m'] = require('./topojson/world_50m.json');
-exports.topojsons['africa_50m'] = require('./topojson/africa_50m.json');
-exports.topojsons['asia_50m'] = require('./topojson/asia_50m.json');
-exports.topojsons['europe_50m'] = require('./topojson/europe_50m.json');
-exports.topojsons['north-america_50m'] = require('./topojson/north-america_50m.json');
-exports.topojsons['south-america_50m'] = require('./topojson/south-america_50m.json');
-exports.topojsons['usa_50m'] = require('./topojson/usa_50m.json');
+exports.topojson = saneTopojson;

dist/plotly.js

@@ -450,14 +450,14 @@ proto.toImage = function() {
 
     var ev = Plotly.Snapshot.toImage(this.graphInfo, {format: format});
 
-    var filename = this.graphInfo.fn || "newplot";
+    var filename = this.graphInfo.fn || 'newplot';
     filename += '.' + format;
 
     ev.once('success', function(result) {
 
         _this._snapshotInProgress = false;
 
-        var downloadLink = document.createElement("a");
+        var downloadLink = document.createElement('a');
         downloadLink.href = result;
         downloadLink.download = filename; // only supported by FF and Chrome
 
@@ -478,6 +478,11 @@ proto.toImage = function() {
     });
 };
 
+proto.sendDataToCloud = function() {
+    var gd = this.graphInfo;
+    Plotly.Plots.sendDataToCloud(gd)
+};
+
 /**
  *
  * @Property config specification hash of button parameters
@@ -547,6 +552,11 @@ proto.config = function config() {
             icon: 'camera',
             click: this.toImage
         },
+        sendDataToCloud: {
+            title: 'save and edit plot in cloud',
+            icon: 'disk',
+            click: this.sendDataToCloud
+        },
         // gl3d
         zoom3d: {
             title: 'Zoom',

dist/plotly.min.js

@@ -1,8 +1,15 @@
 .plotly-notifier {
+    font-family: 'Open Sans'; // Because not all contexts have this specified.
     position: fixed;
     top: 50px;
     right: 20px;
     z-index: 10000;
+    font-size: 10pt;
+    max-width: 180px;
+
+    p {
+        margin: 0;
+    }
 
     .notifier-note {
         min-width: 180px;
@@ -37,5 +44,3 @@
         }
     }
 }
-
-

src/assets/topojson/africa_110m.json renamed to dist/topojson/africa_110m.json

@@ -39,3 +39,4 @@
 .ploticon-3d_rotate:before { content: '\e80c'; } /* '' */
 .ploticon-camera:before { content: '\e80d'; } /* '' */
 .ploticon-movie:before { content: '\e80e'; } /* '' */
+.ploticon-disk:before { content: '\e80f'; } /* '' */

src/assets/topojson/africa_50m.json renamed to dist/topojson/africa_50m.json

@@ -25,6 +25,7 @@ exports.Plots = Plotly.Plots;
 exports.Fx = Plotly.Fx;
 exports.Snapshot = Plotly.Snapshot;
 exports.PlotSchema = Plotly.PlotSchema;
+exports.Queue = Plotly.Queue;
 
 // export d3 used in the bundle
 exports.d3 = require('d3');

src/assets/topojson/asia_110m.json renamed to dist/topojson/asia_110m.json

@@ -1,5 +1,7 @@
 'use strict';
 
+/* global $:false */
+
 var EventEmitter = require('events').EventEmitter;
 
 var Events = {

src/assets/topojson/asia_50m.json renamed to dist/topojson/asia_50m.json

@@ -13,11 +13,8 @@ topojsonUtils.getTopojsonName = function(geoLayout) {
     ].join('');
 };
 
-topojsonUtils.getTopojsonPath = function(topojsonName) {
-    var topojsonUrl = (window.PLOTLYENV &&
-                       window.PLOTLYENV.TOPOJSON_URL) || './topojson/';
-
-    return topojsonUrl + topojsonName + '.json';
+topojsonUtils.getTopojsonPath = function(topojsonURL, topojsonName) {
+    return topojsonURL + topojsonName + '.json';
 };
 
 topojsonUtils.getTopojsonFeatures = function(trace, topojson) {

src/assets/topojson/europe_110m.json renamed to dist/topojson/europe_110m.json

@@ -455,10 +455,10 @@ function plotGeo(gd) {
 
     var i, geoId, fullGeoData, geo;
 
-    // if plotlyjs-geo-assets-bundle is not included,
-    // initialize object to keep reference to every loaded topojsons
+    // if 'plotly-geo-assets.js' is not included,
+    // initialize object to keep reference to every loaded topojson
     if(window.PlotlyGeoAssets === undefined) {
-        window.PlotlyGeoAssets = { topojsons : {} };
+        window.PlotlyGeoAssets = { topojson : {} };
     }
 
     for (i = 0; i < geoIds.length; i++) {
@@ -471,7 +471,8 @@ function plotGeo(gd) {
             geo = new Plotly.Geo(
                 {
                     id: geoId,
-                    container: fullLayout._geocontainer.node()
+                    container: fullLayout._geocontainer.node(),
+                    topojsonURL: gd._context.topojsonURL
                 },
                 fullLayout
             );
@@ -1358,7 +1359,9 @@ Plotly.extendTraces = function extendTraces (gd, update, indices, maxPoints) {
     Plotly.redraw(gd);
 
     var undoArgs = [gd, undo.update, indices, undo.maxPoints];
-    Plotly.Queue.add(gd, Plotly.prependTraces, undoArgs, extendTraces, arguments);
+    if (Plotly.Queue) {
+        Plotly.Queue.add(gd, Plotly.prependTraces, undoArgs, extendTraces, arguments);
+    }
 };
 
 Plotly.prependTraces  = function prependTraces (gd, update, indices, maxPoints) {
@@ -1383,7 +1386,9 @@ Plotly.prependTraces  = function prependTraces (gd, update, indices, maxPoints)
     Plotly.redraw(gd);
 
     var undoArgs = [gd, undo.update, indices, undo.maxPoints];
-    Plotly.Queue.add(gd, Plotly.extendTraces, undoArgs, prependTraces, arguments);
+    if (Plotly.Queue) {
+        Plotly.Queue.add(gd, Plotly.extendTraces, undoArgs, prependTraces, arguments);
+    }
 };
 
 /**
@@ -1427,7 +1432,7 @@ Plotly.addTraces = function addTraces (gd, traces, newIndices) {
     // i.e., we can simply redraw and be done
     if (typeof newIndices === 'undefined') {
         Plotly.redraw(gd);
-        Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
+        if (Plotly.Queue) Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
         return;
     }
 
@@ -1450,10 +1455,10 @@ Plotly.addTraces = function addTraces (gd, traces, newIndices) {
 
     // if we're here, the user has defined specific places to place the new traces
     // this requires some extra work that moveTraces will do
-    Plotly.Queue.startSequence(gd);
-    Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
+    if (Plotly.Queue) Plotly.Queue.startSequence(gd);
+    if (Plotly.Queue) Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
     Plotly.moveTraces(gd, currentIndices, newIndices);
-    Plotly.Queue.stopSequence(gd);
+    if (Plotly.Queue) Plotly.Queue.stopSequence(gd);
 };
 
 /**
@@ -1493,7 +1498,8 @@ Plotly.deleteTraces = function deleteTraces (gd, indices) {
     }
 
     Plotly.redraw(gd);
-    Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
+
+    if (Plotly.Queue) Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
 };
 
 /**
@@ -1587,8 +1593,10 @@ Plotly.moveTraces = function moveTraces (gd, currentIndices, newIndices) {
     }
 
     gd.data = newData;
+
     Plotly.redraw(gd);
-    Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
+
+    if (Plotly.Queue) Plotly.Queue.add(gd, undoFunc, undoArgs, redoFunc, redoArgs);
 };
 
 // -----------------------------------------------------
@@ -2025,7 +2033,9 @@ Plotly.restyle = function restyle(gd, astr, val, traces) {
 
     // now all attribute mods are done, as are redo and undo
     // so we can save them
-    Plotly.Queue.add(gd, restyle, [gd, undoit, traces], restyle, [gd, redoit, traces]);
+    if(Plotly.Queue) {
+        Plotly.Queue.add(gd, restyle, [gd, undoit, traces], restyle, [gd, redoit, traces]);
+    }
 
     // do we need to force a recalc?
     var autorangeOn = false;
@@ -2403,7 +2413,9 @@ Plotly.relayout = function relayout(gd, astr, val) {
     }
     // now all attribute mods are done, as are
     // redo and undo so we can save them
-    Plotly.Queue.add(gd, relayout, [gd, undoit], relayout, [gd, redoit]);
+    if(Plotly.Queue) {
+        Plotly.Queue.add(gd, relayout, [gd, undoit], relayout, [gd, redoit]);
+    }
 
     // calculate autosizing - if size hasn't changed,
     // will remove h&w so we don't need to redraw

src/assets/topojson/europe_50m.json renamed to dist/topojson/europe_50m.json

@@ -35,7 +35,7 @@ module.exports = {
     showTips: true,
 
     // link to open this plot in plotly
-    showLink: true,
+    showLink: false,
 
     // if we show a link, does it contain data or just link to a plotly file?
     sendData: true,
@@ -57,7 +57,10 @@ module.exports = {
 
     // function to add the background color to a different container
     // or 'opaque' to ensure there's white behind it
-    setBackground: defaultSetBackground
+    setBackground: defaultSetBackground,
+
+    // URL to topojson files used in geo charts
+    topojsonURL: 'https://cdn.plot.ly/'
 
 };
 

src/assets/topojson/north-america_110m.json renamed to dist/topojson/north-america_110m.json

@@ -1319,15 +1319,15 @@ fx.modeBar = function(gd){
 function chooseModebarButtons(fullLayout) {
     if(fullLayout._hasGL3D) {
         return [
-            ['toImage'],
+            ['toImage', 'sendDataToCloud'],
             ['orbitRotation', 'tableRotation', 'zoom3d', 'pan3d'],
             ['resetCameraDefault3d', 'resetCameraLastSave3d'],
             ['hoverClosest3d']
         ];
     }
     else if(fullLayout._hasGeo) {
         return [
-            ['toImage'],
+            ['toImage', 'sendDataToCloud'],
             ['zoomInGeo', 'zoomOutGeo', 'resetGeo'],
             ['hoverClosestGeo']
         ];
@@ -1345,9 +1345,9 @@ function chooseModebarButtons(fullLayout) {
         }
     }
 
-    if(allFixed) buttons = [['toImage']];
+    if(allFixed) buttons = [['toImage', 'sendDataToCloud']];
     else buttons = [
-        ['toImage'],
+        ['toImage', 'sendDataToCloud'],
         ['zoom2d', 'pan2d'],
         ['zoomIn2d', 'zoomOut2d', 'resetScale2d', 'autoScale2d']
     ];

src/assets/topojson/north-america_50m.json renamed to dist/topojson/north-america_50m.json

@@ -22,6 +22,7 @@ function Geo(options, fullLayout) {
 
     this.id = options.id;
     this.container = options.container;
+    this.topojsonURL = options.topojsonURL;
 
     // add a few projection types to d3.geo,
     // a subset of https://github.com/d3/d3-geo-projection
@@ -73,27 +74,27 @@ proto.plot = function(geoData, fullLayout) {
 
     _this.framework
         .call(_this.zoom)
-        .on('dblclick', _this.zoomReset);
-
-    // N.B. the on 'dblclick doesn't in d3 3.5+
-    // https://github.com/mbostock/d3/issues/1985
+        .on('dblclick.zoom', _this.zoomReset);
 
     topojsonNameNew = topojsonUtils.getTopojsonName(geoLayout);
 
     if(_this.topojson===null || topojsonNameNew!==_this.topojsonName) {
         _this.topojsonName = topojsonNameNew;
 
-        if(PlotlyGeoAssets.topojsons[_this.topojsonName] !== undefined) {
-            _this.topojson = PlotlyGeoAssets.topojsons[_this.topojsonName];
+        if(PlotlyGeoAssets.topojson[_this.topojsonName] !== undefined) {
+            _this.topojson = PlotlyGeoAssets.topojson[_this.topojsonName];
             _this.onceTopojsonIsLoaded(geoData, geoLayout);
         }
         else {
-            topojsonPath = topojsonUtils.getTopojsonPath(_this.topojsonName);
+            topojsonPath = topojsonUtils.getTopojsonPath(
+                _this.topojsonURL,
+                _this.topojsonName
+            );
 
             // N.B this is async
             d3.json(topojsonPath, function(error, topojson) {
                 _this.topojson = topojson;
-                PlotlyGeoAssets.topojsons[_this.topojsonName] = topojson;
+                PlotlyGeoAssets.topojson[_this.topojsonName] = topojson;
                 _this.onceTopojsonIsLoaded(geoData, geoLayout);
             });
         }