From 4b68fa3caef1346f673f87dd897c00110ab50702 Mon Sep 17 00:00:00 2001 From: Corey Johnson Date: Tue, 5 Jun 2012 11:53:29 -0700 Subject: [PATCH] Remove detach recursion from autocomplete cancel. miniEditor.focusOut was causing cancel to be called multiple times. --- src/extensions/autocomplete.coffee | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/extensions/autocomplete.coffee b/src/extensions/autocomplete.coffee index 8391a4dd1..2f3023bd0 100644 --- a/src/extensions/autocomplete.coffee +++ b/src/extensions/autocomplete.coffee @@ -100,7 +100,8 @@ class Autocomplete extends View attach: -> @confirmed = false - @miniEditor.on 'focusout', => @cancel() unless @confirmed + @miniEditor.on 'focusout', => + @cancel() unless @confirmed @originalSelectedText = @editor.getSelectedText() @originalSelectionBufferRange = @editor.getSelection().getBufferRange() @@ -116,9 +117,10 @@ class Autocomplete extends View @miniEditor.focus() detach: -> + @miniEditor.off("focusout") + super @editor.off(".autocomplete") @editor.focus() - super @miniEditor.buffer.setText('') setPosition: (originalCursorPosition) ->