Changeset 13158

Timestamp:

Jan 14, 2012, 2:28:54 AM (9 years ago)

Author:

livings124

Message:

#4713 (Lion) animate removing rows from the tracker table

Location:

trunk/macosx

Files:

• InfoTrackersViewController.m (modified) (8 diffs)
• TrackerNode.h (modified) (1 diff)
• TrackerNode.m (modified) (1 diff)

trunk/macosx/InfoTrackersViewController.m

@@ -24 +24 @@
 
 #import "InfoTrackersViewController.h"
+#import "NSApplicationAdditions.h"
 #import "Torrent.h"
 #import "TrackerCell.h"
@@ -103 +104 @@
     if ([fTrackerTable editedRow] == -1)
     {
-        [fTrackers release];
+        NSArray * oldTrackers = fTrackers;
         
         if ([fTorrents count] == 1)
@@ -115 +116 @@
         
         [fTrackerTable setTrackers: fTrackers];
-        [fTrackerTable reloadData];
+        
+        if ([NSApp isOnLionOrBetter] && [fTrackers isEqualToArray: oldTrackers])
+            [fTrackerTable setNeedsDisplay: YES];
+        else
+            [fTrackerTable reloadData];
+        
+        [oldTrackers release];
     }
     else
@@ -304 +311 @@
 {
     NSMutableDictionary * removeIdentifiers = [NSMutableDictionary dictionaryWithCapacity: [fTorrents count]];
-    NSUInteger removeCount = 0;
+    NSUInteger removeTrackerCount = 0;
        
     NSIndexSet * selectedIndexes = [fTrackerTable selectedRowIndexes];
     BOOL groupSelected = NO;
+    NSUInteger groupRowIndex = NSNotFound;
+    NSMutableIndexSet * removeIndexes = [NSMutableIndexSet indexSet];
     for (NSUInteger i = 0; i < [fTrackers count]; ++i)
     {
@@ -324 +333 @@
                 
                 [removeSet addObject: [(TrackerNode *)object fullAnnounceAddress]];
-                ++removeCount;
+                ++removeTrackerCount;
+                
+                [removeIndexes addIndex: i];
             }
+            else
+                groupRowIndex = NSNotFound; //don't remove the group row
         }
         else
         {
+            //mark the previous group row for removal, if necessary
+            if (groupRowIndex != NSNotFound)
+                [removeIndexes addIndex: groupRowIndex];
+            
             groupSelected = [selectedIndexes containsIndex: i];
             if (!groupSelected && i > [selectedIndexes lastIndex])
+            {
+                groupRowIndex = NSNotFound;
                 break;
-        }
-    }
+            }
+            
+            groupRowIndex = i;
+        }
+    }
+    
+    //mark the last group for removal, too
+    if (groupRowIndex != NSNotFound)
+        [removeIndexes addIndex: groupRowIndex];
+    
+    NSAssert2(removeTrackerCount <= [removeIndexes count], @"Marked %ld trackers to remove, but only removing %ld rows", removeTrackerCount, [removeIndexes count]);
     
     //we might have no trackers if remove right after a failed add (race condition ftw)
     #warning look into having a failed add apply right away, so that this can become an assert
-    if (removeCount == 0)
+    if (removeTrackerCount == 0)
         return;
     
@@ -344 +372 @@
         NSAlert * alert = [[NSAlert alloc] init];
         
-        if (removeCount > 1)
+        if (removeTrackerCount > 1)
         {
             [alert setMessageText: [NSString stringWithFormat: NSLocalizedString(@"Are you sure you want to remove %d trackers?",
-                                                                "Remove trackers alert -> title"), removeCount]];
+                                                                "Remove trackers alert -> title"), removeTrackerCount]];
             [alert setInformativeText: NSLocalizedString(@"Once removed, Transmission will no longer attempt to contact them."
                                         " This cannot be undone.", "Remove trackers alert -> message")];
@@ -371 +399 @@
             return;
     }
+    
+    
+    if ([NSApp isOnLionOrBetter])
+        [fTrackerTable beginUpdates];
     
     for (Torrent * torrent in removeIdentifiers)
@@ -381 +413 @@
         [fTrackers addObjectsFromArray: [torrent allTrackerStats]];
     
-    [fTrackerTable setTrackers: fTrackers];
-    [fTrackerTable reloadData];
-    [fTrackerTable deselectAll: self];
+    if ([NSApp isOnLionOrBetter])
+    {
+        [fTrackerTable removeRowsAtIndexes: removeIndexes withAnimation: NSTableViewAnimationSlideLeft];
+        
+        [fTrackerTable setTrackers: fTrackers];
+        
+        [fTrackerTable endUpdates];
+    }
+    else
+    {
+        [fTrackerTable setTrackers: fTrackers];
+        
+        [fTrackerTable reloadData];
+        [fTrackerTable deselectAll: self];
+    }
     
     [[NSNotificationCenter defaultCenter] postNotificationName: @"UpdateUI" object: nil]; //incase sort by tracker

trunk/macosx/TrackerNode.h

@@ -37 +37 @@
 - (id) initWithTrackerStat: (tr_tracker_stat *) stat torrent: (Torrent *) torrent;
 
+- (BOOL) isEqual: (id) object;
+
 - (NSString *) host;
 - (NSString *) fullAnnounceAddress;

trunk/macosx/TrackerNode.m

@@ -48 +48 @@
     //this object is essentially immutable after initial setup
     return [self retain];
+}
+
+- (BOOL) isEqual: (id) object
+{
+    if (self == object)
+        return YES;
+    
+    if (![object isKindOfClass: [self class]])
+        return NO;
+    
+    return [self tier] == [object tier] && [[self fullAnnounceAddress] isEqualToString: [object fullAnnounceAddress]];
 }