From d7d3f174aea0ca8fdc162cd1cc0d2533753cf95b Mon Sep 17 00:00:00 2001 From: John Firebaugh Date: Wed, 13 Feb 2013 16:53:40 -0800 Subject: [PATCH] Lasso only nodes for now (fixes #734) Given the (lack of) operations available on mixed node/way selections, this seems just as good as selecting contained ways as well. --- js/id/behavior/lasso.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/js/id/behavior/lasso.js b/js/id/behavior/lasso.js index 6196cbf2e..728b8ad15 100644 --- a/js/id/behavior/lasso.js +++ b/js/id/behavior/lasso.js @@ -49,7 +49,9 @@ iD.behavior.Lasso = function(context) { .on('mouseup.lasso', null); if (d3.event.clientX !== pos[0] || d3.event.clientY !== pos[1]) { - var selected = context.graph().intersects(extent); + var selected = context.graph().intersects(extent).filter(function (entity) { + return entity.type === 'node'; + }); if (selected.length) { context.enter(iD.modes.Select(context, _.pluck(selected, 'id')));