From f0346e7af72748c94915989c1cf7299e899d4803 Mon Sep 17 00:00:00 2001 From: Tom MacWright Date: Sat, 26 Jan 2013 15:32:24 -0500 Subject: [PATCH] Hide nudge menu to keep people from thinking it will sort layers --- js/id/ui/layerswitcher.js | 27 ++++++++++++++++++++++++--- 1 file changed, 24 insertions(+), 3 deletions(-) diff --git a/js/id/ui/layerswitcher.js b/js/id/ui/layerswitcher.js index 7555bd5b3..282bd75e2 100644 --- a/js/id/ui/layerswitcher.js +++ b/js/id/ui/layerswitcher.js @@ -149,14 +149,36 @@ iD.ui.layerswitcher = function(map) { map.redraw(); } - adjustments.selectAll('button') + adjustments.append('a') + .text('▶ fix misalignment') + .attr('href', '#') + .classed('expanded', false) + .on('click', function() { + var exp = d3.select(this).classed('expanded'); + if (!exp) { + nudge_container.style('display', 'block'); + d3.select(this).text('▼ fix misalignment'); + } else { + nudge_container.style('display', 'none'); + d3.select(this).text('▶ fix misalignment'); + } + d3.select(this).classed('expanded', !exp); + d3.event.preventDefault(); + }); + + var nudge_container = adjustments + .append('div') + .attr('class', 'nudge-container') + .style('display', 'none'); + + nudge_container.selectAll('button') .data(directions).enter() .append('button') .attr('class', 'nudge') .text(function(d) { return d[0]; }) .on('click', nudge); - adjustments.append('button') + nudge_container.append('button') .text('reset') .attr('class', 'reset') .on('click', function() { @@ -164,7 +186,6 @@ iD.ui.layerswitcher = function(map) { map.redraw(); }); - selection.call(clickoutside); selectLayer(map.background.source()); }