fix delay after clicking close button (#857)

package.json

@@ -45,7 +45,7 @@
   "dependencies": {
     "@babel/runtime": "^7.0.0",
     "mapbox-gl": "^1.0.0",
-    "mjolnir.js": "^2.1.2",
+    "mjolnir.js": "^2.2.0",
     "prop-types": "^15.7.2",
     "react-virtualized-auto-sizer": "^1.0.2",
     "viewport-mercator-project": "^6.1.0"

src/components/popup.js

@@ -164,11 +164,18 @@ export default class Popup extends BaseControl<PopupProps, *, HTMLDivElement> {
       evt.stopPropagation();
     }
 
-    if (
-      evt.type === 'click' &&
-      (this.props.closeOnClick || evt.target.className === 'mapboxgl-popup-close-button')
-    ) {
+    if (this.props.closeOnClick || evt.target.className === 'mapboxgl-popup-close-button') {
       this.props.onClose();
+
+      const {eventManager} = this._context;
+      if (eventManager) {
+        // Using with InteractiveMap
+        // After we call `onClose` on `anyclick`, this component will be unmounted
+        // at which point we unregister the event listeners and stop blocking propagation.
+        // Then after a short delay a `click` event will fire
+        // Attach a one-time event listener here to prevent it from triggering `onClick` of the base map
+        eventManager.once('click', e => e.stopPropagation(), evt.target);
+      }
     }
   };
 

yarn.lock

@@ -6326,10 +6326,10 @@ mixin-deep@^1.2.0:
     for-in "^1.0.2"
     is-extendable "^1.0.1"
 
-mjolnir.js@^2.1.2:
-  version "2.1.2"
-  resolved "https://registry.yarnpkg.com/mjolnir.js/-/mjolnir.js-2.1.2.tgz#db89819c418b6f3d200b1c51386fc7487f98bd5a"
-  integrity sha512-OjvkNzTCdJd/I7XxgIomNozYS6me6OWY+t8I6SZlINaFmNmgAW3I5x+Hd8sgA+WxAjFvBf576lD7N1MGo89/WA==
+mjolnir.js@^2.2.0:
+  version "2.2.0"
+  resolved "https://registry.yarnpkg.com/mjolnir.js/-/mjolnir.js-2.2.0.tgz#3ca4a1e18321595e86bb9c8b5f68da8f5f68fcae"
+  integrity sha512-nh58odNDNms8TnajKU+BYYtiffIVsxe5yg4CP/rDwJWTrNKR38ULf5CdjMWkWFSsoocHL157yQH8PEuNNU0kvA==
   dependencies:
     "@babel/runtime" "^7.0.0"
     hammerjs "^2.0.8"