From 9a654ae8fa31037887717fe9d530b91a5bab4893 Mon Sep 17 00:00:00 2001 From: Jon Atkins Date: Mon, 18 Nov 2013 07:33:35 +0000 Subject: [PATCH] bookmarks plugin - fix auto draw to pass the right options (line/polygon) - otherwise geodesic lines can show a fill due to the curve - add note about API needed in draw tools plugin for this to use --- plugins/bookmarks-by-zaso.user.js | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/plugins/bookmarks-by-zaso.user.js b/plugins/bookmarks-by-zaso.user.js index 86b5ef44..ef694fcc 100644 --- a/plugins/bookmarks-by-zaso.user.js +++ b/plugins/bookmarks-by-zaso.user.js @@ -616,12 +616,13 @@ }); if(latlngs.length >= 2 && latlngs.length <= 3) { + // TODO: add an API to draw-tools rather than assuming things about it's internals var newItem; - // var options = {color:"#a24ac3",weight:4,opacity:.5} - var options = window.plugin.drawTools.polygonOptions; - - if(latlngs.length == 3) { newItem = L.geodesicPolygon(latlngs, options); } - else if(latlngs.length == 2) { newItem = L.geodesicPolyline(latlngs, options); } + if(latlngs.length == 2) { + newItem = L.geodesicPolyline(latlngs, window.plugin.drawTools.lineOptions); + } else { + newItem = L.geodesicPolygon(latlngs, window.plugin.drawTools.polygonOptions); + } $('#bkmrksAutoDrawer a.bkmrk.selected').removeClass('selected'); newItem.addTo(window.plugin.drawTools.drawnItems);