Minor improvement to bounds handling. Publish next version.

Author: rowanwins

CHANGES.md

@@ -1,3 +1,7 @@
+## 0.0.7
+- Make the `useSeviceBounds` option non-blocking so that data draws quicker and only the bounds check only gets applied to subsequent requests 
+- Update build process and packaging
+
 ## 0.0.6
 - Fix conditional on fetchOptions 
 

docs/src/Map.vue

@@ -6,6 +6,7 @@
 import { Map, Popup } from 'maplibre-gl'
 import 'maplibre-gl/dist/maplibre-gl.css'
 import FeatureService from '../../src/main'
+// import FeatureService from '../../dist/mapbox-gl-arcgis-featureserver.js'
 
 let layer = null
 let map = null

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mapbox-gl-arcgis-featureserver",
-  "version": "0.0.6",
+  "version": "0.0.7",
   "description": "A small library for requesting features from an arcgis feature server using tiles",
   "main": "dist/mapbox-gl-arcgis-featureserver.js",
   "module": "dist/mapbox-gl-arcgis-featureserver.esm.js",
@@ -15,7 +15,7 @@
     "build-docs": "cross-env NODE_ENV=production webpack --config docs/webpack.config.js --mode production --progress --hide-modules",
     "dev": "cross-env webpack-dev-server --config docs/webpack.config.js --mode development --open --hot"
   },
-  "author": "",
+  "author": "Rowan Winsemius",
   "license": "Apache-2.0",
   "dependencies": {
     "@mapbox/tilebelt": "^1.0.2",
@@ -48,5 +48,18 @@
     "webpack": "^4.29.0",
     "webpack-cli": "^3.2.1",
     "webpack-dev-server": "^3.1.14"
-  }
+  },
+  "repository": "github:rowanwins/mapbox-gl-arcgis-featureserver",
+  "bugs": {
+    "url": "https://github.com/rowanwins/mapbox-gl-arcgis-featureserver/issues"
+  },
+  "keywords": [
+    "mapbox-gl-js",
+    "mapbox-gl",
+    "maplibre-gl-js",
+    "maplibre-gl",
+    "esri",
+    "arcgis",
+    "featureserver"
+  ]
 }

rollup.config.js

@@ -9,22 +9,19 @@ const output = (input, file, format, plugins) => ({
     file,
     format
   },
-  external: ['pbf'],
   plugins
 })
 
 export default [
   output('./src/main.js', './dist/mapbox-gl-arcgis-featureserver.js', 'umd', [
-    commonjs(),
-    resolve()
+    commonjs()
   ]),
   output('./src/main.js', './dist/mapbox-gl-arcgis-featureserver.min.js', 'umd', [
-    commonjs(),
     resolve(),
+    commonjs(),
     terser()
   ]),
   output('./src/main.js', './dist/mapbox-gl-arcgis-featureserver.esm.js', 'esm', [
-    commonjs(),
-    resolve()
+    commonjs()
   ])
 ]

src/main.js

@@ -54,7 +54,6 @@ export default class FeatureService {
           const serviceExtent = this.serviceMetadata.extent
           if (serviceExtent.spatialReference.wkid === 4326) {
             this._setBounds([serviceExtent.xmin, serviceExtent.ymin, serviceExtent.xmax, serviceExtent.ymax])
-            this._clearAndRefreshTiles()
           } else {
             this._projectBounds()
           }
@@ -66,6 +65,7 @@ export default class FeatureService {
 
         this._setAttribution()
         this.enableRequests()
+        this._clearAndRefreshTiles()
       })
   }
 
@@ -163,8 +163,10 @@ export default class FeatureService {
     const bounds = this._map.getBounds().toArray()
     const primaryTile = tilebelt.bboxToTile([bounds[0][0], bounds[0][1], bounds[1][0], bounds[1][1]])
 
-    if (this._esriServiceOptions.useSeviceBounds && !this._doesTileOverlapBbox(this._maxExtent, bounds)) {
-      return
+    if (this._esriServiceOptions.useSeviceBounds) {
+      if (this._maxExtent[0] !== -Infinity && !this._doesTileOverlapBbox(this._maxExtent, bounds)) {
+        return
+      }
     }
 
     // If we're not using a static zoom level we'll round to the nearest even zoom level
@@ -399,7 +401,6 @@ export default class FeatureService {
       .then((data) => {
         const extent = data.geometries[0]
         this._maxExtent = [extent.xmin, extent.ymin, extent.xmax, extent.ymax]
-        this._clearAndRefreshTiles()
       })
       .catch((error) => {
         // if projection endpoint has already been set to fallback, do not re-request project bounds