Opening multiple documents will now use the same window size

If frame isn't explicitly set using the autosaveName then the autosaveName is only used for the first two windows. Opening the third and subsequent windows then start using the default window size instead of the autosave frame.

Author: ksuther

Classes/Controllers/PBGitWindowController.m

@@ -46,8 +46,10 @@ - (void)synchronizeWindowTitleWithDocumentName
 {
     [super synchronizeWindowTitleWithDocumentName];
 
-    // Point window proxy icon at project directory, not internal .git dir
-    [[self window] setRepresentedURL:self.repository.workingDirectoryURL];
+	if ([self isWindowLoaded]) {
+		// Point window proxy icon at project directory, not internal .git dir
+		[[self window] setRepresentedURL:self.repository.workingDirectoryURL];
+	}
 }
 
 - (void)windowWillClose:(NSNotification *)notification
@@ -99,9 +101,14 @@ - (BOOL) validateMenuItem:(NSMenuItem *)menuItem remoteTitle:(NSString *)localis
 }
 
 
-- (void) awakeFromNib
+- (void) windowDidLoad
 {
-	[[self window] setDelegate:self];
+	[super windowDidLoad];
+
+	// Explicitly set the frame using the autosave name
+	// Opening the first and second documents works fine, but the third and subsequent windows aren't positioned correctly
+	[[self window] setFrameUsingName:@"GitX"];
+	[[self window] setRepresentedURL:self.repository.workingDirectoryURL];
 
 	sidebarController = [[PBGitSidebarController alloc] initWithRepository:repository superController:self];
 	[[sidebarController view] setFrame:[sourceSplitView bounds]];
@@ -110,8 +117,6 @@ - (void) awakeFromNib
 
 	[[statusField cell] setBackgroundStyle:NSBackgroundStyleRaised];
 	[progressIndicator setUsesThreadedAnimation:YES];
-
-	[self showWindow:nil];
 }
 
 - (void) removeAllContentSubViews

Resources/English.lproj/RepositoryWindow.xib

@@ -136,6 +136,9 @@
                     <constraint firstAttribute="bottom" secondItem="396" secondAttribute="bottom" id="pIe-nQ-ntf"/>
                 </constraints>
             </view>
+            <connections>
+                <outlet property="delegate" destination="-2" id="N8Y-0Q-xZn"/>
+            </connections>
         </window>
     </objects>
     <resources>