Question

I have a NSWindowController with a setCurrentView method in which I use the following code to resize the window to the new view’s size.

[self resizeWindowForContentSize: [newView frame].size];

NSView *contentView = [[self window] contentView];
NSDictionary *ani = [NSDictionary dictionaryWithObject:transition 
                                                forKey:@"subviews"];
[contentView setAnimations:ani];    
[[contentView animator] replaceSubview:currentView with:newView];

Edit: Added the resizeWindowForContentSize I found on CocoaDev.

- (void)resizeWindowForContentSize:(NSSize) size {
    NSRect windowFrame = [NSWindow contentRectForFrameRect:[[self window] frame]
        styleMask:[[self window] styleMask]];
    NSRect newWindowFrame = [NSWindow frameRectForContentRect: 
        NSMakeRect( NSMinX( windowFrame ), NSMaxY( windowFrame ) - size.height, size.width, size.height )
    styleMask:[[self window] styleMask]];
    [[self window] setFrame:newWindowFrame display:YES animate:[[self window] isVisible]];
}

The problem I’m having is that as you switch between views (using NSToolbar) the views begin to move around in the contentView. Over time they actually draw further and further away until eventually there is just a blank view.

Answer 1

Instead of using the class methods:

+[NSWindow frameRectForContentRect:styleMask:]
+[NSWindow contentRectForFrameRect:styleMask:]

use the instance methods:

-[NSWindow frameRectForContentRect:]
-[NSWindow contentRectForFrameRect:]

instead. The instance methods take toolbars into account, and should be used whenever you have an NSWindow instance — which you do via [self window].

For example, assuming that’s the problem with your -resizeWindowForContentSize: method:

- (void)resizeWindowForContentSize:(NSSize) size {
    NSWindow *window = [self window];

    NSRect windowFrame = [window contentRectForFrameRect:[window frame]];
    NSRect newWindowFrame = [window frameRectForContentRect:
        NSMakeRect( NSMinX( windowFrame ), NSMaxY( windowFrame ) - size.height, size.width, size.height )];
    [window setFrame:newWindowFrame display:YES animate:[window isVisible]];
}