Merge pull request #2105 from CartoDB/13808-ie-popup-with-image

Better popups with just images on IE and Edge
CartoDB · Apr 24, 2018 · 2354d72 · 2354d72
2 parents 3a17e62 + 6210eab
commit 2354d72
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "carto.js",
-  "version": "4.0.1-0",
+  "version": "4.0.1-13808",
   "description": "CARTO javascript library",
   "repository": {
     "type": "git",

diff --git a/src/geo/ui/infowindow-view.js b/src/geo/ui/infowindow-view.js
@@ -312,13 +312,22 @@ var Infowindow = View.extend({
     return false;
   },
 
+  _imageOnHookNotSupported: function () {
+    var noCssClipPath = util.ie || util.browser.ie || util.browser.edge;
+    return noCssClipPath;
+  },
+
   _loadImageHook: function (imageDimensions, coverDimensions, url) {
     var $hook = this.$('.CDB-hook');
-
     if (!$hook) {
       return;
     }
 
+    if (this._imageOnHookNotSupported()) {
+      $hook.hide();
+      return;
+    }
+
     var $hookImage = $('<img />')
       .addClass('CDB-hookImage js-hookImage')
       .attr('src', url);