- Fix tab bars in windows with only one tab not disappearing after dragging a tab past/through them, by using PSMTabBar notifications instead of our own notifications; also fix PSMTabBar notification ordering.

 - When a tab drag starts, reorder all windows in front of other applications

2 files changed, 17 insertions, 6 deletions

diff --git a/Source/SPAppController.m b/Source/SPAppController.m
index 2d719c95..c5c2ed2d 100644
--- a/Source/SPAppController.m
+++ b/Source/SPAppController.m
@@ -33,6 +33,7 @@
 #import "SPConstants.h"
 #import "SPWindowController.h"
 
+#import <PSMTabBar/PSMTabBarControl.h>
 #import <Sparkle/Sparkle.h>
 
 @implementation SPAppController
@@ -79,7 +80,10 @@
 	
 	// Register SPAppController for AppleScript events
 	[[NSScriptExecutionContext sharedScriptExecutionContext] setTopLevelObject:self];
-	
+
+	// Register for drag start notifications - used to bring all windows to front
+	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(tabDragStarted:) name:PSMTabDragDidBeginNotification object:nil];
+
 	isNewFavorite = NO;
 }
 
@@ -500,6 +504,14 @@
 	return nil;
 }
 
+/**
+ * When tab drags start, bring all the windows in front of other applications.
+ */
+- (void)tabDragStarted:(id)sender
+{
+	[NSApp arrangeInFront:self];
+}
+
 #pragma mark -
 #pragma mark IBAction methods
 
@@ -920,6 +932,8 @@
  */
 - (void)dealloc
 {
+	[[NSNotificationCenter defaultCenter] removeObserver:self];
+
 	if(_spfSessionDocData) [_spfSessionDocData release], _spfSessionDocData = nil;
 	[prefsController release], prefsController = nil;
 	[aboutController release], aboutController = nil;
diff --git a/Source/SPWindowController.m b/Source/SPWindowController.m
index ed443ac4..7eb83765 100644
--- a/Source/SPWindowController.m
+++ b/Source/SPWindowController.m
@@ -73,8 +73,8 @@
 	closeTabMenuItem = [[[[NSApp mainMenu] itemWithTag:SPMainMenuFile] submenu] itemWithTag:1103];
 
 	// Register for drag start and stop notifications - used to show/hide tab bars
-	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(tabDragStarted:) name:@"SPTabDragStart" object:nil];
-	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(tabDragStopped:) name:@"SPTabDragStop" object:nil];
+	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(tabDragStarted:) name:PSMTabDragDidBeginNotification object:nil];
+	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(tabDragStopped:) name:PSMTabDragDidEndNotification object:nil];
 }
 
 /**
@@ -375,7 +375,6 @@
  */
 - (BOOL)tabView:(NSTabView *)aTabView shouldDragTabViewItem:(NSTabViewItem *)tabViewItem fromTabBar:(PSMTabBarControl *)tabBarControl
 {
-	[[NSNotificationCenter defaultCenter] postNotificationName:@"SPTabDragStart" object:self];
 	return YES;
 }
 
@@ -408,8 +407,6 @@
 
 	// Check the window and move it to front if it's key (eg for new window creation)
 	if ([[tabBarControl window] isKeyWindow]) [[tabBarControl window] orderFront:self];
-
-	[[NSNotificationCenter defaultCenter] postNotificationName:@"SPTabDragStop" object:self];
 }
 
 /**