Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit bc848b6

Browse files
committed
Merge pull request #4782 from zaggino/issue4781
Fixes issue 4781.
2 parents 14accd9 + 35ee28e commit bc848b6

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

src/search/FindReplace.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ define(function (require, exports, module) {
289289
var doReplaceConfirm = Strings.CMD_REPLACE +
290290
'? <button id="replace-yes" class="btn">' + Strings.BUTTON_YES +
291291
'</button> <button id="replace-no" class="btn">' + Strings.BUTTON_NO +
292-
'</button> <button class="btn">' + Strings.BUTTON_STOP + '</button>';
292+
'</button> <button id="replace-stop" class="btn">' + Strings.BUTTON_STOP + '</button>';
293293

294294
function replace(editor, all) {
295295
var cm = editor._codeMirror;
@@ -343,6 +343,9 @@ define(function (require, exports, module) {
343343
doReplace(match);
344344
} else if (e.target.id === "replace-no") {
345345
advance();
346+
} else if (e.target.id === "replace-stop") {
347+
// Destroy modalBar on stop
348+
modalBar = null;
346349
}
347350
});
348351
};

0 commit comments

Comments
 (0)