From 0ee5d6f933e104058961be7330370470c047824e Mon Sep 17 00:00:00 2001 From: Liz Looney Date: Sat, 27 Aug 2016 22:03:06 -0700 Subject: [PATCH] Check result of window.confirm before deleting variables. --- core/workspace.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/core/workspace.js b/core/workspace.js index db7415753..a8e97c04b 100644 --- a/core/workspace.js +++ b/core/workspace.js @@ -322,9 +322,12 @@ Blockly.Workspace.prototype.deleteVariable = function(name) { return; } } - window.confirm( + var ok = window.confirm( Blockly.Msg.DELETE_VARIABLE_CONFIRMATION.replace('%1', uses.length). replace('%2', name)); + if (!ok) { + return; + } } Blockly.Events.setGroup(true);