ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

No need to adjust hitTolerance for pixel ratio

Browse Source
This commit is contained in:
Andreas Hocevar
2020-11-27 11:52:52 +01:00
parent 7b5f53753b
commit 5659397b39
1 changed files with 4 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ol/PluggableMap.js
+4 -10
View File
@@ -75,7 +75,7 @@ import {removeNode} from './dom.js';
* {@link module:ol/layer/Layer layer-candidate} and it should return a boolean value. * {@link module:ol/layer/Layer layer-candidate} and it should return a boolean value.
* Only layers which are visible and for which this function returns `true` * Only layers which are visible and for which this function returns `true`
* will be tested for features. By default, all visible layers will be tested. * will be tested for features. By default, all visible layers will be tested.
* @property {number} [hitTolerance=0] Hit-detection tolerance in pixels. Pixels * @property {number} [hitTolerance=0] Hit-detection tolerance in css pixels. Pixels
* inside the radius around the given position will be checked for features. * inside the radius around the given position will be checked for features.
* @property {boolean} [checkWrapped=true] Check-Wrapped Will check for for wrapped geometries inside the range of * @property {boolean} [checkWrapped=true] Check-Wrapped Will check for for wrapped geometries inside the range of
* +/- 1 world width. Works only if a projection is used that can be wrapped. * +/- 1 world width. Works only if a projection is used that can be wrapped.
@@ -559,9 +559,7 @@ class PluggableMap extends BaseObject {
const coordinate = this.getCoordinateFromPixelInternal(pixel); const coordinate = this.getCoordinateFromPixelInternal(pixel);
opt_options = opt_options !== undefined ? opt_options : {}; opt_options = opt_options !== undefined ? opt_options : {};
const hitTolerance = const hitTolerance =
opt_options.hitTolerance !== undefined opt_options.hitTolerance !== undefined ? opt_options.hitTolerance : 0;
? opt_options.hitTolerance * this.frameState_.pixelRatio
: 0;
const layerFilter = const layerFilter =
opt_options.layerFilter !== undefined ? opt_options.layerFilter : TRUE; opt_options.layerFilter !== undefined ? opt_options.layerFilter : TRUE;
const checkWrapped = opt_options.checkWrapped !== false; const checkWrapped = opt_options.checkWrapped !== false;
@@ -624,9 +622,7 @@ class PluggableMap extends BaseObject {
} }
const options = opt_options || {}; const options = opt_options || {};
const hitTolerance = const hitTolerance =
options.hitTolerance !== undefined options.hitTolerance !== undefined ? options.hitTolerance : 0;
? options.hitTolerance * this.frameState_.pixelRatio
: 0;
const layerFilter = options.layerFilter || TRUE; const layerFilter = options.layerFilter || TRUE;
return this.renderer_.forEachLayerAtPixel( return this.renderer_.forEachLayerAtPixel(
pixel, pixel,
@@ -654,9 +650,7 @@ class PluggableMap extends BaseObject {
const layerFilter = const layerFilter =
opt_options.layerFilter !== undefined ? opt_options.layerFilter : TRUE; opt_options.layerFilter !== undefined ? opt_options.layerFilter : TRUE;
const hitTolerance = const hitTolerance =
opt_options.hitTolerance !== undefined opt_options.hitTolerance !== undefined ? opt_options.hitTolerance : 0;
? opt_options.hitTolerance * this.frameState_.pixelRatio
: 0;
const checkWrapped = opt_options.checkWrapped !== false; const checkWrapped = opt_options.checkWrapped !== false;
return this.renderer_.hasFeatureAtCoordinate( return this.renderer_.hasFeatureAtCoordinate(
coordinate, coordinate,