diff --git a/spec/text-editor-spec.coffee b/spec/text-editor-spec.coffee index 1f667c217..59b8a218e 100644 --- a/spec/text-editor-spec.coffee +++ b/spec/text-editor-spec.coffee @@ -1729,7 +1729,7 @@ describe "TextEditor", -> expect(editor.getSelectedBufferRanges()).toEqual [[[0, 0], [0, 3]]] describe ".consolidateSelections()", -> - it "destroys all selections but the most recent, returning true if any selections were destroyed", -> + it "destroys all selections but the least recent, returning true if any selections were destroyed", -> editor.setSelectedBufferRange([[3, 16], [3, 21]]) selection1 = editor.getLastSelection() selection2 = editor.addSelectionForBufferRange([[3, 25], [3, 34]]) @@ -1737,10 +1737,10 @@ describe "TextEditor", -> expect(editor.getSelections()).toEqual [selection1, selection2, selection3] expect(editor.consolidateSelections()).toBeTruthy() - expect(editor.getSelections()).toEqual [selection3] - expect(selection3.isEmpty()).toBeFalsy() + expect(editor.getSelections()).toEqual [selection1] + expect(selection1.isEmpty()).toBeFalsy() expect(editor.consolidateSelections()).toBeFalsy() - expect(editor.getSelections()).toEqual [selection3] + expect(editor.getSelections()).toEqual [selection1] describe "when the cursor is moved while there is a selection", -> makeSelection = -> selection.setBufferRange [[1, 2], [1, 5]] diff --git a/src/text-editor.coffee b/src/text-editor.coffee index b09e8cd33..f8f147862 100644 --- a/src/text-editor.coffee +++ b/src/text-editor.coffee @@ -2241,11 +2241,11 @@ class TextEditor extends Model @consolidateSelections() @getLastSelection().clear(options) - # Reduce multiple selections to the most recently added selection. + # Reduce multiple selections to the least recently added selection. consolidateSelections: -> selections = @getSelections() if selections.length > 1 - selection.destroy() for selection in selections[0...-1] + selection.destroy() for selection in selections[1...(selections.length)] true else false