From 90308b3a75d799931a0123360588c76114b4c299 Mon Sep 17 00:00:00 2001 From: Ian B Date: Sat, 16 Feb 2013 14:58:20 +0100 Subject: [PATCH] Fix missing OK button on confirm modal (#791) --- js/id/ui/confirm.js | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/js/id/ui/confirm.js b/js/id/ui/confirm.js index 177bb94f0..1de510c5c 100644 --- a/js/id/ui/confirm.js +++ b/js/id/ui/confirm.js @@ -1,17 +1,23 @@ iD.ui.confirm = function(selection) { var modal = iD.ui.modal(selection); modal.select('.modal').classed('modal-alert', true); - modal.select('.content') + var section = modal.select('.content') .attr('class','modal-section fillD') - .append('div') + + var description = section.append('div') .attr('class', 'description'); - var nochanges = modal.select('.content') - .append('button') - .attr('class','action centered') + + var buttonwrap = section.append('div') + .attr('class', 'buttons cf'); + + var okbutton = buttonwrap.append('button') + .attr('class', 'action centered') .on('click.confirm', function() { modal.remove(); }); - nochanges.append('span').attr('class','label').text('Okay'); + + okbutton.append('span').attr('class','icon apply icon-pre-text'); + okbutton.append('span').attr('class','label').text('Okay'); return modal; };