From df98483b065a61cdd3b941ce2d86e4c94bfd595b Mon Sep 17 00:00:00 2001 From: Derek Hammer Date: Wed, 27 Jan 2010 01:13:19 -0500 Subject: [PATCH] Removing extraneous selection did change notifications --- AppKit/CPTableView.j | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/AppKit/CPTableView.j b/AppKit/CPTableView.j index ef8b36f68..1019bbe9b 100644 --- a/AppKit/CPTableView.j +++ b/AppKit/CPTableView.j @@ -658,6 +658,9 @@ window.setTimeout(function(){ { if (([rows firstIndex] != CPNotFound && [rows firstIndex] < 0) || [rows lastIndex] >= [self numberOfRows]) return; + + if ([rows isEqualToIndexSet:_selectedRowIndexes]) + return; // We deselect all columns when selecting rows. if ([_selectedColumnIndexes count] > 0) @@ -2261,10 +2264,6 @@ window.setTimeout(function(){ } } - if (![_previouslySelectedRowIndexes isEqualToIndexSet:_selectedRowIndexes]) - [self _noteSelectionDidChange]; - - if (mouseIsUp && (_implementedDataSourceMethods & CPTableViewDataSource_tableView_setObjectValue_forTableColumn_row_) && !_trackingPointMovedOutOfClickSlop @@ -2660,7 +2659,6 @@ window.setTimeout(function(){ if(i) { [self scrollRowToVisible:i]; - [self _noteSelectionDidChange]; } } @@ -2710,7 +2708,6 @@ window.setTimeout(function(){ if(i) { [self scrollRowToVisible:i]; - [self _noteSelectionDidChange]; } }