From bac39d3996e06c28ec73c13dbc91d0ae78a53e93 Mon Sep 17 00:00:00 2001 From: Pierre GIRAUD Date: Fri, 5 Aug 2016 11:22:59 +0200 Subject: [PATCH] Fix cursor change in translate interaction --- src/ol/interaction/translate.js | 22 +++++----------------- 1 file changed, 5 insertions(+), 17 deletions(-) diff --git a/src/ol/interaction/translate.js b/src/ol/interaction/translate.js index e0530949e0..85e11a9b5c 100644 --- a/src/ol/interaction/translate.js +++ b/src/ol/interaction/translate.js @@ -215,30 +215,18 @@ ol.interaction.Translate.handleDragEvent_ = function(event) { */ ol.interaction.Translate.handleMoveEvent_ = function(event) { var elem = event.map.getTargetElement(); - var intersectingFeature = event.map.forEachFeatureAtPixel(event.pixel, - function(feature) { - return feature; - }); - - if (intersectingFeature) { - var isSelected = false; - - if (this.features_ && - ol.array.includes(this.features_.getArray(), intersectingFeature)) { - isSelected = true; - } + // Change the cursor to grab/grabbing if hovering any of the features managed + // by the interaction + if (this.featuresAtPixel_(event.pixel, event.map)) { this.previousCursor_ = elem.style.cursor; - // WebKit browsers don't support the grab icons without a prefix elem.style.cursor = this.lastCoordinate_ ? - '-webkit-grabbing' : (isSelected ? '-webkit-grab' : 'pointer'); + '-webkit-grabbing' : '-webkit-grab'; // Thankfully, attempting to set the standard ones will silently fail, // keeping the prefixed icons - elem.style.cursor = !this.lastCoordinate_ ? - 'grabbing' : (isSelected ? 'grab' : 'pointer'); - + elem.style.cursor = this.lastCoordinate_ ? 'grabbing' : 'grab'; } else { elem.style.cursor = this.previousCursor_ !== undefined ? this.previousCursor_ : '';