Update plots to work as standalone modules

Author: mdtusz

src/plots/cartesian/index.js

@@ -17,3 +17,13 @@ exports.attr = ['xaxis', 'yaxis'];
 exports.idRoot = ['x', 'y'];
 
 exports.attributes = require('./attributes');
+
+exports.idRegex = {
+    x: /^x([2-9]|[1-9][0-9]+)?$/,
+    y: /^y([2-9]|[1-9][0-9]+)?$/
+};
+
+exports.attrRegex = {
+    x: /^x([2-9]|[1-9][0-9]+)?$/,
+    y: /^y([2-9]|[1-9][0-9]+)?$/
+};

src/plots/cartesian/layout_attributes.js

@@ -7,13 +7,11 @@
 */
 
 'use strict';
-
-var Plotly = require('../../plotly');
+var Cartesian = require('./index');
 var fontAttrs = require('../font_attributes');
 var colorAttrs = require('../../components/color/attributes');
 var extendFlat = require('../../lib/extend').extendFlat;
 
-
 module.exports = {
     title: {
         valType: 'string',
@@ -385,8 +383,8 @@ module.exports = {
         valType: 'enumerated',
         values: [
             'free',
-            Plotly.Plots.subplotsRegistry.cartesian.idRegex.x.toString(),
-            Plotly.Plots.subplotsRegistry.cartesian.idRegex.y.toString()
+            Cartesian.idRegex.x.toString(),
+            Cartesian.idRegex.y.toString()
         ],
         role: 'info',
         description: [
@@ -414,8 +412,8 @@ module.exports = {
         valType: 'enumerated',
         values: [
             'free',
-            Plotly.Plots.subplotsRegistry.cartesian.idRegex.x.toString(),
-            Plotly.Plots.subplotsRegistry.cartesian.idRegex.y.toString()
+            Cartesian.idRegex.x.toString(),
+            Cartesian.idRegex.y.toString()
         ],
         role: 'info',
         description: [

src/plots/geo/geo.js

@@ -11,10 +11,13 @@
 
 /* global PlotlyGeoAssets:false */
 
-var Plotly = require('../../plotly');
 var d3 = require('d3');
 
+var Color = require('../../components/color');
+var Drawing = require('../../components/drawing');
+
 var Plots = require('../../plots/plots');
+var Axes = require('../../plots/cartesian/axes');
 
 var addProjectionsToD3 = require('./projections');
 var createGeoScale = require('./set_scale');
@@ -350,7 +353,7 @@ function styleFillLayer(selection, layerName, geoLayout) {
     selection.select('.' + layerName)
         .selectAll('path')
             .attr('stroke', 'none')
-            .call(Plotly.Color.fill, geoLayout[layerAdj + 'color']);
+            .call(Color.fill, geoLayout[layerAdj + 'color']);
 }
 
 function styleLineLayer(selection, layerName, geoLayout) {
@@ -359,16 +362,16 @@ function styleLineLayer(selection, layerName, geoLayout) {
     selection.select('.' + layerName)
         .selectAll('path')
             .attr('fill', 'none')
-            .call(Plotly.Color.stroke, geoLayout[layerAdj + 'color'])
-            .call(Plotly.Drawing.dashLine, '', geoLayout[layerAdj + 'width']);
+            .call(Color.stroke, geoLayout[layerAdj + 'color'])
+            .call(Drawing.dashLine, '', geoLayout[layerAdj + 'width']);
 }
 
 function styleGraticule(selection, axisName, geoLayout) {
     selection.select('.' + axisName + 'graticule')
         .selectAll('path')
             .attr('fill', 'none')
-            .call(Plotly.Color.stroke, geoLayout[axisName].gridcolor)
-            .call(Plotly.Drawing.dashLine, '', geoLayout[axisName].gridwidth);
+            .call(Color.stroke, geoLayout[axisName].gridcolor)
+            .call(Drawing.dashLine, '', geoLayout[axisName].gridwidth);
 }
 
 proto.styleLayer = function(selection, layerName, geoLayout) {
@@ -451,10 +454,10 @@ function createMockAxis(fullLayout) {
     var mockAxis = {
         type: 'linear',
         showexponent: 'all',
-        exponentformat: Plotly.Axes.layoutAttributes.exponentformat.dflt,
+        exponentformat: Axes.layoutAttributes.exponentformat.dflt,
         _td: { _fullLayout: fullLayout }
     };
 
-    Plotly.Axes.setConvert(mockAxis);
+    Axes.setConvert(mockAxis);
     return mockAxis;
 }

src/plots/geo/index.js

@@ -9,11 +9,9 @@
 
 'use strict';
 
-var Plotly = require('../../plotly');
-
 var Geo = require('./geo');
 
-var Plots = Plotly.Plots;
+var Plots = require('../../plots/plots');
 
 
 exports.name = 'geo';
@@ -22,6 +20,10 @@ exports.attr = 'geo';
 
 exports.idRoot = 'geo';
 
+exports.idRegex = /^geo([2-9]|[1-9][0-9]+)?$/;
+
+exports.attrRegex = /^geo([2-9]|[1-9][0-9]+)?$/;
+
 exports.attributes = require('./layout/attributes');
 
 exports.layoutAttributes = require('./layout/layout_attributes');

src/plots/geo/layout/axis_defaults.js

@@ -9,7 +9,7 @@
 
 'use strict';
 
-var Plotly = require('../../../plotly');
+var Lib = require('../../../lib');
 var constants = require('../../../constants/geo_constants');
 var axisAttributes = require('./axis_attributes');
 
@@ -20,7 +20,7 @@ module.exports = function supplyGeoAxisLayoutDefaults(geoLayoutIn, geoLayoutOut)
     var axisIn, axisOut;
 
     function coerce(attr, dflt) {
-        return Plotly.Lib.coerce(axisIn, axisOut, axisAttributes, attr, dflt);
+        return Lib.coerce(axisIn, axisOut, axisAttributes, attr, dflt);
     }
 
     function getRangeDflt(axisName) {
@@ -55,7 +55,7 @@ module.exports = function supplyGeoAxisLayoutDefaults(geoLayoutIn, geoLayoutOut)
 
         var range = coerce('range', rangeDflt);
 
-        Plotly.Lib.noneOrAll(axisIn.range, axisOut.range, [0, 1]);
+        Lib.noneOrAll(axisIn.range, axisOut.range, [0, 1]);
 
         coerce('tick0', range[0]);
         coerce('dtick', axisName==='lonaxis' ? 30 : 10);

src/plots/geo/layout/defaults.js

@@ -9,20 +9,21 @@
 
 'use strict';
 
-var Plotly = require('../../../plotly');
+var Lib = require('../../../lib');
+var Plots = require('../../plots');
 var constants = require('../../../constants/geo_constants');
 var layoutAttributes = require('./layout_attributes');
 var supplyGeoAxisLayoutDefaults = require('./axis_defaults');
 
 
 module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
-    var geos = Plotly.Plots.getSubplotIdsInData(fullData, 'geo'),
+    var geos = Plots.getSubplotIdsInData(fullData, 'geo'),
         geosLength = geos.length;
 
     var geoLayoutIn, geoLayoutOut;
 
     function coerce(attr, dflt) {
-        return Plotly.Lib.coerce(geoLayoutIn, geoLayoutOut, layoutAttributes, attr, dflt);
+        return Lib.coerce(geoLayoutIn, geoLayoutOut, layoutAttributes, attr, dflt);
     }
 
     for(var i = 0; i < geosLength; i++) {

src/plots/gl2d/index.js

@@ -22,6 +22,16 @@ exports.attr = ['xaxis', 'yaxis'];
 
 exports.idRoot = ['x', 'y'];
 
+exports.idRegex = {
+    x: /^x([2-9]|[1-9][0-9]+)?$/,
+    y: /^y([2-9]|[1-9][0-9]+)?$/
+};
+
+exports.attrRegex = {
+    x: /^xaxis([2-9]|[1-9][0-9]+)?$/,
+    y: /^yaxis([2-9]|[1-9][0-9]+)?$/
+};
+
 exports.attributes = require('../cartesian/attributes');
 
 exports.plot = function plotGl2d(gd) {

src/plots/gl3d/index.js

@@ -21,6 +21,10 @@ exports.attr = 'scene';
 
 exports.idRoot = 'scene';
 
+exports.idRegex = /^scene([2-9]|[1-9][0-9]+)?$/;
+
+exports.attrRegex = /^scene([2-9]|[1-9][0-9]+)?$/;
+
 exports.attributes = require('./layout/attributes');
 
 exports.layoutAttributes = require('./layout/layout_attributes');

src/plots/plots.js

@@ -41,7 +41,7 @@ plots.fontWeight = 'normal';
  */
 plots.register = function(_module, thisType, categoriesIn, meta) {
     if(modules[thisType]) {
-        console.warn('type ' + thisType + ' already registered');
+        console.log('type ' + thisType + ' already registered');
         return;
     }
 
@@ -132,32 +132,12 @@ plots.registerSubplot = function(_module) {
     var plotType = _module.name;
 
     if(subplotsRegistry[plotType]) {
-        throw new Error('plot type' + plotType + ' already registered');
-    }
-
-    var attr = _module.attr,
-        idRoot = _module.idRoot;
-
-    var regexStart = '^',
-        regexEnd = '([2-9]|[1-9][0-9]+)?$',
-        hasXY = (plotType === 'cartesian' || plotType === 'gl2d');
-
-    function makeRegex(mid) {
-        return new RegExp(regexStart + mid + regexEnd);
+        console.log('plot type ' + plotType + ' already registered');
+        return;
     }
 
     // not sure what's best for the 'cartesian' type at this point
     subplotsRegistry[plotType] = _module;
-
-    // register the regex representing the set of all valid attribute names
-    subplotsRegistry[plotType].attrRegex = hasXY ?
-        { x: makeRegex(attr[0]), y: makeRegex(attr[1]) } :
-        makeRegex(attr);
-
-    // register the regex representing the set of all valid attribute ids
-    subplotsRegistry[plotType].idRegex = hasXY ?
-        { x: makeRegex(idRoot[0]), y: makeRegex(idRoot[1]) } :
-        makeRegex(idRoot);
 };
 
 // TODO separate the 'find subplot' step (which looks in layout)