add mapbox-gl CSS when registering mapbox basePlotModule

... to avoid console warning on subplot instantiation

Author: etpinard

src/plots/mapbox/constants.js

@@ -42,6 +42,7 @@ module.exports = {
     // a subset of node_modules/mapbox-gl/dist/mapbox-gl.css
     styleRules: {
         map: 'overflow:hidden;position:relative;',
-        'missing-css': 'display:none',
+        'missing-css': 'display:none;',
+        'canary': 'background-color:salmon;'
     }
 };

src/plots/mapbox/index.js

@@ -15,13 +15,10 @@ var getSubplotCalcData = require('../../plots/get_data').getSubplotCalcData;
 var xmlnsNamespaces = require('../../constants/xmlns_namespaces');
 
 var createMapbox = require('./mapbox');
-var constants = require('./constants');
 
 var MAPBOX = 'mapbox';
 
-for(var k in constants.styleRules) {
-    Lib.addStyleRule('.mapboxgl-' + k, constants.styleRules[k]);
-}
+var constants = exports.constants = require('./constants');
 
 exports.name = MAPBOX;
 

src/registry.js

@@ -12,6 +12,7 @@ var Loggers = require('./lib/loggers');
 var noop = require('./lib/noop');
 var pushUnique = require('./lib/push_unique');
 var isPlainObject = require('./lib/is_plain_object');
+var addStyleRule = require('./lib/dom').addStyleRule;
 var ExtendModule = require('./lib/extend');
 
 var basePlotAttributes = require('./plots/attributes');
@@ -267,6 +268,15 @@ function registerTraceModule(_module) {
     if(_module.layoutAttributes) {
         extendFlat(exports.traceLayoutAttributes, _module.layoutAttributes);
     }
+
+    // add mapbox-gl CSS here to avoid console warning on instantiation
+    var basePlotModule = _module.basePlotModule;
+    if(basePlotModule.name === 'mapbox') {
+        var styleRules = basePlotModule.constants.styleRules;
+        for(var k in styleRules) {
+            addStyleRule('.mapboxgl-' + k, styleRules[k]);
+        }
+    }
 }
 
 function registerSubplot(_module) {