diff options
author | rowanbeentje <rowan@beent.je> | 2010-03-25 00:33:42 +0000 |
---|---|---|
committer | rowanbeentje <rowan@beent.je> | 2010-03-25 00:33:42 +0000 |
commit | 5061643c1bf92a3583e4ede059951e1f17befac9 (patch) | |
tree | 431bdc27b6c5b37cd91f8b0d96954a551f047c91 /Source | |
parent | 3b70bf3ed475602fac435b86ab564423ae1dfdb0 (diff) | |
download | sequelpro-5061643c1bf92a3583e4ede059951e1f17befac9.tar.gz sequelpro-5061643c1bf92a3583e4ede059951e1f17befac9.tar.bz2 sequelpro-5061643c1bf92a3583e4ede059951e1f17befac9.zip |
- Correctly store scroll and selection for the navigation history when filtering content tables
Diffstat (limited to 'Source')
-rw-r--r-- | Source/SPHistoryController.m | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/Source/SPHistoryController.m b/Source/SPHistoryController.m index aae33e81..dd302969 100644 --- a/Source/SPHistoryController.m +++ b/Source/SPHistoryController.m @@ -211,6 +211,8 @@ if (contentSortCol) [contentState setObject:contentSortCol forKey:@"sortCol"]; if (contentSelectedIndexSet) [contentState setObject:contentSelectedIndexSet forKey:@"selection"]; if (contentFilter) [contentState setObject:contentFilter forKey:@"filter"]; + + // Update the table content states with this information - used when switching tables to restore last used view. [tableContentStates setObject:contentState forKey:[NSString stringWithFormat:@"%@.%@", [theDatabase backtickQuotedString], [theTable backtickQuotedString]]]; } @@ -219,7 +221,7 @@ [history removeObjectsInRange:NSMakeRange(historyPosition + 1, [history count] - historyPosition - 1)]; } else if (historyPosition != NSNotFound && historyPosition == [history count] - 1) { - NSDictionary *currentHistoryEntry = [history objectAtIndex:historyPosition]; + NSMutableDictionary *currentHistoryEntry = [history objectAtIndex:historyPosition]; // If the table is the same, and the filter settings haven't changed, delete the // last entry so it can be replaced. This updates navigation within a table, rather than @@ -236,6 +238,17 @@ { [history removeLastObject]; + // If the only db/table/view are the same, but the filter settings have changed, also store the + // position details on the *previous* history item + } else if ([[currentHistoryEntry objectForKey:@"database"] isEqualToString:theDatabase] + && [[currentHistoryEntry objectForKey:@"table"] isEqualToString:theTable] + && ([[currentHistoryEntry objectForKey:@"view"] integerValue] == theView + || ((![currentHistoryEntry objectForKey:@"contentFilter"] && contentFilter) + || ![[currentHistoryEntry objectForKey:@"contentFilter"] isEqualToDictionary:contentFilter]))) + { + [currentHistoryEntry setObject:[NSValue valueWithRect:contentViewport] forKey:@"contentViewport"]; + if (contentSelectedIndexSet) [currentHistoryEntry setObject:contentSelectedIndexSet forKey:@"contentSelectedIndexSet"]; + // Special case: if the last history item is currently active, and has no table, // but the new selection does - delete the last entry, in order to replace it. // This improves history flow. @@ -260,7 +273,7 @@ if (contentFilter) [newEntry setObject:contentFilter forKey:@"contentFilter"]; [history addObject:newEntry]; - + // If there are now more than fifty history entries, remove one from the start if ([history count] > 50) [history removeObjectAtIndex:0]; |