diff --git a/lib/OpenLayers/Control/TransformFeature.js b/lib/OpenLayers/Control/TransformFeature.js index 437124bfcf..571963146c 100644 --- a/lib/OpenLayers/Control/TransformFeature.js +++ b/lib/OpenLayers/Control/TransformFeature.js @@ -166,6 +166,15 @@ OpenLayers.Control.TransformFeature = OpenLayers.Class(OpenLayers.Control, { */ dragControl: null, + /** + * APIProperty: irregular + * {Boolean} Make scaling/resizing work irregularly. If true then + * dragging a handle causes the feature to resize in the direction + * of movement. If false then the feature resizes symetrically + * about it's center. + */ + irregular: false, + /** * Constructor: OpenLayers.Control.TransformFeature * Create a new transform feature control. @@ -390,6 +399,10 @@ OpenLayers.Control.TransformFeature = OpenLayers.Class(OpenLayers.Control, { var dy1 = this.y - centerGeometry.y; var dx0 = dx1 - (this.x - oldGeom.x); var dy0 = dy1 - (this.y - oldGeom.y); + if (control.irregular && !control._setfeature) { + dx1 -= (this.x - oldGeom.x) / 2; + dy1 -= (this.y - oldGeom.y) / 2; + } this.x = oldX; this.y = oldY; var scale, ratio = 1; @@ -411,6 +424,13 @@ OpenLayers.Control.TransformFeature = OpenLayers.Class(OpenLayers.Control, { control.box.geometry.resize(scale, centerGeometry, ratio); control.box.geometry.rotate(control.rotation, centerGeometry); control.transformFeature({scale: scale, ratio: ratio}); + if (control.irregular && !control._setfeature) { + var newCenter = centerGeometry.clone(); + newCenter.x += Math.abs(oldX - centerGeometry.x) < 0.00001 ? 0 : (this.x - oldX); + newCenter.y += Math.abs(oldY - centerGeometry.y) < 0.00001 ? 0 : (this.y - oldY); + control.box.geometry.move(this.x - oldX, this.y - oldY); + control.transformFeature({center: newCenter}); + } }; // Override for rotation handle move - make sure that the box and diff --git a/tests/Control/TransformFeature.html b/tests/Control/TransformFeature.html index 00fb0a90b4..3279867876 100644 --- a/tests/Control/TransformFeature.html +++ b/tests/Control/TransformFeature.html @@ -82,6 +82,44 @@ control.box.move(center); t.geom_eq(control.handles[0].geometry, control.box.geometry.components[0], "handle moved with box"); } + + function test_handleMove(t) { + t.plan(16); + var map = new OpenLayers.Map("map", {allOverlays: true}); + var layer = new OpenLayers.Layer.Vector(); + var feature = new OpenLayers.Feature.Vector( + OpenLayers.Geometry.fromWKT("POLYGON((-1 -1, 1 -1, 1 1, -1 1))")); + layer.addFeatures([feature]); + map.addLayer(layer); + map.setCenter(new OpenLayers.LonLat(0, 0), 18); + var control = new OpenLayers.Control.TransformFeature(layer); + map.addControl(control); + control.setFeature(feature); + + var bottomLeft = new OpenLayers.LonLat(-2, -2); + control.handles[0].move(bottomLeft); + t.geom_eq(control.handles[0].geometry, new OpenLayers.Geometry.Point(-2, -2), "bottom left handle at -2,-2"); + t.geom_eq(control.handles[1].geometry, new OpenLayers.Geometry.Point(0, -2), "bottom handle at 0,-2"); + t.geom_eq(control.handles[2].geometry, new OpenLayers.Geometry.Point(2, -2), "bottom right handle at 2,-2"); + t.geom_eq(control.handles[3].geometry, new OpenLayers.Geometry.Point(2, 0), "right handle at 2,0"); + t.geom_eq(control.handles[4].geometry, new OpenLayers.Geometry.Point(2, 2), "top right handle at 2,2"); + t.geom_eq(control.handles[5].geometry, new OpenLayers.Geometry.Point(0, 2), "top handle at 0,2"); + t.geom_eq(control.handles[6].geometry, new OpenLayers.Geometry.Point(-2, 2), "top left handle at -2,2"); + t.geom_eq(control.handles[7].geometry, new OpenLayers.Geometry.Point(-2, 0), "left handle at -2,0"); + + control.irregular = true; + + var bottomLeft = new OpenLayers.LonLat(-3, -3); + control.handles[0].move(bottomLeft); + t.geom_eq(control.handles[0].geometry, new OpenLayers.Geometry.Point(-3, -3), "bottom left handle at -3,-3"); + t.geom_eq(control.handles[1].geometry, new OpenLayers.Geometry.Point(-0.5, -3), "bottom handle at 0,-3"); + t.geom_eq(control.handles[2].geometry, new OpenLayers.Geometry.Point(2, -3), "bottom right handle at 2,-3"); + t.geom_eq(control.handles[3].geometry, new OpenLayers.Geometry.Point(2, -0.5), "right handle at 2,0"); + t.geom_eq(control.handles[4].geometry, new OpenLayers.Geometry.Point(2, 2), "top right handle at 2,2"); + t.geom_eq(control.handles[5].geometry, new OpenLayers.Geometry.Point(-0.5, 2), "top handle at 0,2"); + t.geom_eq(control.handles[6].geometry, new OpenLayers.Geometry.Point(-3, 2), "top left handle at -3,2"); + t.geom_eq(control.handles[7].geometry, new OpenLayers.Geometry.Point(-3, -0.5), "left handle at -3,0"); + } diff --git a/tests/Test.AnotherWay.geom_eq.js b/tests/Test.AnotherWay.geom_eq.js index 3e3ad8486f..893c5b5a2f 100644 --- a/tests/Test.AnotherWay.geom_eq.js +++ b/tests/Test.AnotherWay.geom_eq.js @@ -34,6 +34,35 @@ } } + /** + * Function assertFloatEqual + * Test two objects for floating point equivalence. Throw an exception + * if not equivalent. + * + * Parameters: + * got - {Object} + * expected - {Object} + * msg - {String} The message to be thrown. This message will be appended + * with ": got {got} but expected {expected}" where got and expected are + * replaced with string representations of the above arguments. + */ + function assertFloatEqual(got, expected, msg) { + var OpenLayers = Test.AnotherWay._g_test_iframe.OpenLayers; + if(got === undefined) { + got = "undefined"; + } else if (got === null) { + got = "null"; + } + if(expected === undefined) { + expected = "undefined"; + } else if (expected === null) { + expected = "null"; + } + if(Math.abs(got - expected) > Math.pow(10, -OpenLayers.Util.DEFAULT_PRECISION)) { + throw msg + ": got '" + got + "' but expected '" + expected + "'"; + } + } + /** * Function assertGeometryEqual * Test two geometries for equivalence. Geometries are considered @@ -58,9 +87,9 @@ if(got instanceof OpenLayers.Geometry.Point) { // compare points - assertEqual(got.x, expected.x, "x mismatch"); - assertEqual(got.y, expected.y, "y mismatch"); - assertEqual(got.z, expected.z, "z mismatch"); + assertFloatEqual(got.x, expected.x, "x mismatch"); + assertFloatEqual(got.y, expected.y, "y mismatch"); + assertFloatEqual(got.z, expected.z, "z mismatch"); } else { // compare components assertEqual( @@ -107,4 +136,4 @@ } } -})(); \ No newline at end of file +})();