Don't use withFrozenRendering in ol.interaction.Select#handleMapBrowserEvent

This commit is contained in:
Tom Payne
2014-02-27 12:51:42 +01:00
parent 73be72380b
commit ad29e6bceb
+36 -42
View File
@@ -97,50 +97,44 @@ ol.interaction.Select.prototype.handleMapBrowserEvent =
var add = this.addCondition_(mapBrowserEvent); var add = this.addCondition_(mapBrowserEvent);
var map = mapBrowserEvent.map; var map = mapBrowserEvent.map;
var features = this.featureOverlay_.getFeatures(); var features = this.featureOverlay_.getFeatures();
map.withFrozenRendering( if (add) {
/** map.forEachFeatureAtPixel(mapBrowserEvent.pixel,
* @this {ol.interaction.Select} /**
*/ * @param {ol.Feature} feature Feature.
function() { * @param {ol.layer.Layer} layer Layer.
if (add) { */
map.forEachFeatureAtPixel(mapBrowserEvent.pixel, function(feature, layer) {
/** if (goog.array.indexOf(features.getArray(), feature) == -1) {
* @param {ol.Feature} feature Feature. features.push(feature);
* @param {ol.layer.Layer} layer Layer.
*/
function(feature, layer) {
if (goog.array.indexOf(features.getArray(), feature) == -1) {
features.push(feature);
}
}, undefined, this.layerFilter_);
} else {
/** @type {ol.Feature|undefined} */
var feature = map.forEachFeatureAtPixel(mapBrowserEvent.pixel,
/**
* @param {ol.Feature} feature Feature.
* @param {ol.layer.Layer} layer Layer.
*/
function(feature, layer) {
return feature;
}, undefined, this.layerFilter_);
if (goog.isDef(feature)) {
if (features.getLength() == 1) {
if (features.getAt(0) !== feature) {
features.setAt(0, feature);
}
} else {
if (features.getLength() != 1) {
features.clear();
}
features.push(feature);
}
} else {
if (features.getLength() !== 0) {
features.clear();
}
} }
}, undefined, this.layerFilter_);
} else {
/** @type {ol.Feature|undefined} */
var feature = map.forEachFeatureAtPixel(mapBrowserEvent.pixel,
/**
* @param {ol.Feature} feature Feature.
* @param {ol.layer.Layer} layer Layer.
*/
function(feature, layer) {
return feature;
}, undefined, this.layerFilter_);
if (goog.isDef(feature)) {
if (features.getLength() == 1) {
if (features.getAt(0) !== feature) {
features.setAt(0, feature);
} }
}, this); } else {
if (features.getLength() != 1) {
features.clear();
}
features.push(feature);
}
} else {
if (features.getLength() !== 0) {
features.clear();
}
}
}
return false; return false;
}; };