Oh please Google, give us some well named methods that solve this issue. We muddle around with obfuscated code trying in vein to solve your pesky setCenter flicker. This brings us a bit closer with even sized viewports but is still embarassingly wrong for users panning across the dateline with odd sized viewport. r=crschmidt (closes #1174)
git-svn-id: http://svn.openlayers.org/trunk/openlayers@5627 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
@@ -239,8 +239,6 @@ OpenLayers.Layer.EventPane = OpenLayers.Class(OpenLayers.Layer, {
|
|||||||
if ( !(newCenter.equals(oldCenter)) ||
|
if ( !(newCenter.equals(oldCenter)) ||
|
||||||
!(newZoom == oldZoom) ) {
|
!(newZoom == oldZoom) ) {
|
||||||
|
|
||||||
var center = this.getMapObjectLonLatFromOLLonLat(newCenter);
|
|
||||||
|
|
||||||
if (dragging && this.dragPanMapObject &&
|
if (dragging && this.dragPanMapObject &&
|
||||||
this.smoothDragPan) {
|
this.smoothDragPan) {
|
||||||
var resolution = this.map.getResolution();
|
var resolution = this.map.getResolution();
|
||||||
@@ -248,6 +246,7 @@ OpenLayers.Layer.EventPane = OpenLayers.Class(OpenLayers.Layer, {
|
|||||||
var dY = (newCenter.lat - oldCenter.lat) / resolution;
|
var dY = (newCenter.lat - oldCenter.lat) / resolution;
|
||||||
this.dragPanMapObject(dX, dY);
|
this.dragPanMapObject(dX, dY);
|
||||||
} else {
|
} else {
|
||||||
|
var center = this.getMapObjectLonLatFromOLLonLat(newCenter);
|
||||||
var zoom = this.getMapObjectZoomFromOLZoom(newZoom);
|
var zoom = this.getMapObjectZoomFromOLZoom(newZoom);
|
||||||
this.setMapObjectCenter(center, zoom, dragging);
|
this.setMapObjectCenter(center, zoom, dragging);
|
||||||
}
|
}
|
||||||
@@ -282,6 +281,18 @@ OpenLayers.Layer.EventPane = OpenLayers.Class(OpenLayers.Layer, {
|
|||||||
var moPixel = this.getMapObjectPixelFromOLPixel(viewPortPx);
|
var moPixel = this.getMapObjectPixelFromOLPixel(viewPortPx);
|
||||||
var moLonLat = this.getMapObjectLonLatFromMapObjectPixel(moPixel);
|
var moLonLat = this.getMapObjectLonLatFromMapObjectPixel(moPixel);
|
||||||
lonlat = this.getOLLonLatFromMapObjectLonLat(moLonLat);
|
lonlat = this.getOLLonLatFromMapObjectLonLat(moLonLat);
|
||||||
|
var xrem = this.map.size.w % 2;
|
||||||
|
var yrem = this.map.size.h % 2;
|
||||||
|
if(xrem != 0 || yrem != 0) {
|
||||||
|
// odd sized viewport
|
||||||
|
var olPx = viewPortPx.add(xrem, yrem);
|
||||||
|
var moPx = this.getMapObjectPixelFromOLPixel(olPx);
|
||||||
|
var moLoc = this.getMapObjectLonLatFromMapObjectPixel(moPx);
|
||||||
|
var olLoc = this.getOLLonLatFromMapObjectLonLat(moLoc);
|
||||||
|
// adjust by half a pixel in odd dimension(s)
|
||||||
|
lonlat.lon += (olLoc.lon - lonlat.lon) / 2;
|
||||||
|
lonlat.lat += (olLoc.lat - lonlat.lat) / 2;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
return lonlat;
|
return lonlat;
|
||||||
},
|
},
|
||||||
|
|||||||
Reference in New Issue
Block a user