Merge pull request #3111 from plotly/dom-layout

support dom element as argument to Plotly.makeTemplate

Author: antoinerg

src/plot_api/template_api.js

@@ -31,7 +31,7 @@ var dfltConfig = require('./plot_config');
  *     `layout.template` in another figure.
  */
 exports.makeTemplate = function(figure) {
-    figure = Lib.extendDeep({_context: dfltConfig}, figure);
+    figure = Lib.extendDeep({_context: dfltConfig}, {data: figure.data, layout: figure.layout});
     Plots.supplyDefaults(figure);
     var data = figure.data || [];
     var layout = figure.layout || {};

test/jasmine/tests/template_test.js

@@ -141,6 +141,33 @@ describe('makeTemplate', function() {
 
         expect(template).toEqual(expected);
     });
+
+    it('works on DOM element', function(done) {
+        var mock = Lib.extendDeep({}, scatterFillMock);
+        var gd = createGraphDiv();
+
+        Plotly.newPlot(gd, mock)
+        .then(function() {
+            var template = Plotly.makeTemplate(gd);
+            delete(template.layout.xaxis);
+            delete(template.layout.yaxis);
+            expect(template).toEqual({
+                data: {scatter: [
+                  {fill: 'tonext', line: {shape: 'spline'}},
+                  {fill: 'tonext'},
+                  {fill: 'toself'}
+                ] },
+                layout: {
+                    title: 'Fill toself and tonext',
+                    width: 400,
+                    height: 400
+                }
+            });
+        })
+      .catch(failTest)
+      .then(destroyGraphDiv)
+      .then(done);
+    });
 });
 
 // statics of template application are all covered by the template mock