OpenLayers OpenLayers

Ticket #531 (closed bug: fixed)

Opened 2 years ago

Last modified 1 year ago

OpenLayer.Control.moveTo bug

Reported by: openlayers Assigned to:
Priority: major Milestone: 2.4 Release
Component: Control Version: SVN
Keywords: Cc: cbarnes@intelligentlight.com
State:

Description

I noticed that setting a control position would set the the x and y position of the control to just the x value. Checking the code in Control.js I noticed the following:

/**

moveTo: function (px) {

if ((px != null) && (this.div != null)) {

this.div.style.left = px.x + "px";

this.div.style.top = px.x + "px"; // <- should be px.y

}

},

When I changed it to px.y as describe control can be positioned correctly.

Craig Barnes cbarnes@intelligentlight.com

Attachments

moveControl.patch (458 bytes) - added by euzuro on 03/13/07 13:02:54.
patch for moveTo fix

Change History

03/13/07 13:02:54 changed by euzuro

  • attachment moveControl.patch added.

patch for moveTo fix

03/13/07 13:09:51 changed by euzuro

  • keywords changed from moveTo to moveTo review.
  • milestone set to 2.4 Release.

03/13/07 13:11:32 changed by euzuro

  • version set to SVN.

03/13/07 14:06:43 changed by sderle

  • status changed from new to closed.
  • resolution set to fixed.

applied as #2726.

07/05/07 17:03:27 changed by euzuro

  • keywords deleted.