Remove direct mapbox-gl dependencies

[mcwhittemore]

Currently mapbox-gl-draw include mapbox-gl as part of its build. This means end users are downloading mapbox-gl twice. Lets find ways to remove these dependencies.

• mapbox-gl-draw/src/api.js

  Line 3 in 33346ac

  import mapboxgl from 'mapbox-gl';

• mapbox-gl-draw/src/util.js

  Line 3 in 85141d0

  var mapboxgl = require('mapbox-gl');

[kelvinabrokwa]

@mcwhittemore for the second one could we just replace mapboxgl.Point with point-geometry?

[lucaswoj]

We should make upstream changes to avoid the need to inherit mapboxgl.Control. Until then you can read from window.mapboxgl, copy-paste the implementation, or require('mapbox-gl/js/ui/control/control')

[mcwhittemore]

@kelvinabrokwa - do we even need a module for this?

@lucaswoj - Are you thinking of breaking Control out of mapbox-gl-js? It feels like this could lead to the just another dependency problem.

[lucaswoj]

Are you thinking of breaking Control out of mapbox-gl-js? It feels like this could lead to the just another dependency problem.

I'm thinking about removing the need to subclass something to create a control. We can move all the logic from Control to Map. See mapbox/mapbox-gl-js#2118