Merge pull request mapbox#537 from mapbox/fix-534

remove truncation from draw
jisaacks · Nov 8, 2016 · 724e3f8 · 724e3f8
2 parents bbfd25c + e902343
commit 724e3f8
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 26 deletions.
diff --git a/src/lib/mode_handler.js b/src/lib/mode_handler.js
@@ -1,4 +1,3 @@
-const truncatePoint = require('./truncate_point');
 
 var ModeHandler = function(mode, DrawContext) {
 
@@ -29,9 +28,6 @@ var ModeHandler = function(mode, DrawContext) {
   };
 
   var delegate = function (eventName, event) {
-    if (event.lngLat) {
-      event.lngLat = truncatePoint(event.lngLat);
-    }
     var handles = handlers[eventName];
     var iHandle = handles.length;
     while (iHandle--) {

diff --git a/src/lib/move_features.js b/src/lib/move_features.js
@@ -1,4 +1,3 @@
-const truncatePoint = require('./truncate_point');
 const xtend = require('xtend');
 const constrainFeatureMovement = require('./constrain_feature_movement');
 const Constants = require('../constants');
@@ -10,10 +9,10 @@ module.exports = function(features, delta) {
     const currentCoordinates = feature.getCoordinates();
 
     const moveCoordinate = (coord) => {
-        let point = truncatePoint({
+        let point = {
           lng: coord[0] + constrainedDelta.lng,
           lat: coord[1] + constrainedDelta.lat
-        });
+        };
         return [point.lng, point.lat];
     };
     const moveRing = (ring) => ring.map(coord => moveCoordinate(coord));

diff --git a/src/lib/truncate_point.js b/src/lib/truncate_point.js
diff --git a/test/move_features.test.js b/test/move_features.test.js
@@ -16,18 +16,6 @@ test('moveFeatures point', t => {
   t.end();
 });
 
-test('moveFeatures point and truncate the Position to 6 decimal points', t => {
-  const point = new Point(mockFeatureContext, getGeoJSON('point'));
-  point.setCoordinates([10, 20]);
-  moveFeatures([point], { lng: 50.000000001, lat: 20.000000001 });
-  t.deepEqual(point.getCoordinates(), [60, 40]);
-
-  point.setCoordinates([10, 20]);
-  moveFeatures([point], { lng: 50.123456789, lat: 20.123456789 });
-  t.deepEqual(point.getCoordinates(), [60.123456, 40.123456]);
-  t.end();
-});
-
 test('moveFeatures point beyond north limit map', t => {
   const point = new Point(mockFeatureContext, getGeoJSON('point'));
   point.setCoordinates([10, 20]);