Before moving View Controllers into Content.xib, Settings.xib, Reports.xib

Author: jerrykrinock

SSYHierarchicalTabViewItem.h

@@ -44,5 +44,5 @@ extern NSString* const constDiscontiguousTabViewHierarchyString ;
  receiver's identifier appended.
  */
 - (BOOL)isDeeplySelected ;
-
+#warning The 'unfortunately' above is not true because -superview returns the tab view  (Not the item, but at least it's something)
 @end

SSYHierarchicalTabViewItem.m

@@ -16,6 +16,33 @@ - (NSTabViewItem*)selectedChild {
             break ;
         }
     }
+
+    //  The following code seems to have been a good guess.  That is,
+    //  it seems to work, but I haven't really thought it through.
+    //  The idea is that it can reach down to find a tab view through, I
+    //  think, one level of NSView subview.  I need this for the topTabView
+    //  in BookMacster.  This "feature" is not explained in the header doc.
+    if (!selectedChild) {
+        for (NSView* subview in [[self view] subviews]) {
+//          NSLog(@"2 Considering subview %@", subview) ;
+            if ([subview respondsToSelector:@selector(subviews)]) {
+                for (NSTabView* innerSubview in [subview subviews]) {
+//                  NSLog(@"3 Considering innerSubview %@", innerSubview) ;
+                    if ([innerSubview isKindOfClass:[NSTabView class]]) {
+                        selectedChild = [innerSubview selectedTabViewItem] ;
+                        // Note that we consider only the *first* subview which is an
+                        // NSTabView.  For this method to make sense, this must also be the
+                        // *only* subview which is an NSTabViewItem.
+                        break ;
+                    }
+                }
+            }
+            
+            if (selectedChild) {
+                break ;
+            }
+        }
+    }
 
     return selectedChild ;
 }
@@ -24,16 +51,20 @@ - (NSTabViewItem*)selectedLeafmostTabViewItem {
     NSTabViewItem* leafItem = self ;
     NSTabViewItem* selectedChild = nil ;
     do {
+        // This is in case the leaf item in a tree of SSYHierarchicalTabViewItem
+        // objects is not itself a class descendant of
+        // SSYHierarchicalTabViewItem.  It is also for safety,
+        // in case someone sends this message to a tab view item
+        // which is not or does not inherit from this class.
         if (![leafItem respondsToSelector:@selector(selectedChild)]) {
             break ;
         }
 
         selectedChild = [(SSYHierarchicalTabViewItem*)leafItem selectedChild] ;
-        if (selectedChild) {
+        if ([selectedChild isKindOfClass:[NSTabViewItem class]]) {
             leafItem = selectedChild ;
         }
     } while (selectedChild != nil) ;
-    /*SSYDBL*/ NSLog(@"<< %s returning leaf = %@ for %@", __PRETTY_FUNCTION__, [leafItem identifier], [self identifier]) ;
 
     return leafItem ;
 }
@@ -65,7 +96,7 @@ - (BOOL)isDeeplySelected {
                   [self identifier]) ;
         }
     }
-    /*SSYDBL*/ NSLog(@"<< %s is returning %hhd for %@", __PRETTY_FUNCTION__, answer, [self identifier]) ;
+    // NSLog(@"<< %s is returning %hhd for %@", __PRETTY_FUNCTION__, answer, [self identifier]) ;
 
     return answer ;
 }

SSYLazyView.h

@@ -2,6 +2,14 @@
 
 @class SSYExtrospectiveViewController ;
 
+/*
+ @details  
+ 
+ In the Xcode xib editor, you may have one or more initial placeholder subviews
+ in your Lazy View.  For example, you may place a text field with large
+ font size that says "Loading Stuff…".  All of these placeholder subviews
+ will be removed when the new view is placed in.
+ */
 @interface SSYLazyView : NSView {
     SSYExtrospectiveViewController* m_viewController ;
     BOOL m_isLoaded ;
@@ -32,6 +40,6 @@
  @brief    Creates the receiver's view controller and loads the receiver's
  view, or if these things have not already been done, no op.
  */
-- (void)load ;
+- (void)loadInWindow:(NSWindow*)window ;
 
 @end

SSYLazyView.m

@@ -9,6 +9,7 @@ @implementation SSYLazyView
 - (id)initWithFrame:(NSRect)frame {
     self = [super initWithFrame:frame] ;
     if (self) {
+        /*SSYDBL*/ NSLog(@"%@ did init", self) ;
         // Initialization code here.
     }
 
@@ -23,43 +24,80 @@ + (NSString*)lazyNibName {
     return @"Internal Error 939-7834" ;
 }
 
-- (void)load {
-#if 11
-#warning Not loading any tabs
-    return ;
-#endif
-    if ([self isLoaded]) {
+- (void)loadInWindow:(NSWindow*)window {
+   if ([self isLoaded]) {
 		return ;
 	}
 
     NSString* nibName = [[self class] lazyNibName] ;
     Class viewControllerClass = [[self class] lazyViewControllerClass] ;
     SSYExtrospectiveViewController* viewCon = [(SSYExtrospectiveViewController*)[viewControllerClass alloc] initWithNibName:nibName
                                                                                                                             bundle:nil] ;
-    /*SSYDBL*/ NSLog(@"%@ will load nib %@ and instantiate %@", [self className], nibName, [viewCon className]) ;
     [self setViewController:viewCon] ;
-    [viewCon setWindow:[self window]] ; // experimental
-//    /*SSYDBL*/ NSLog(@"7651 %@ has view %@ in window %@", viewCon, [viewCon view], [viewCon window]) ;
+    [viewCon setWindow:window] ; // experimental
     [viewCon release] ;
     [viewCon loadView] ;
 
-    [self addSubview:[viewCon view]] ;
+    // Remove any placeholder subviews.  In BookMacster,  there are two…
+    // (0) An SSYSizeFixxerSubview.
+    // (1) A text field with a string that says "Loading <Something>…"
+    NSInteger nSubviews = [[self subviews] count] ;
+    for (NSInteger i=(nSubviews-1); i>=0; i--) {
+        NSView* subview = [[self subviews] objectAtIndex:i] ;
+        [subview removeFromSuperviewWithoutNeedingDisplay] ;
+    }
+    
+    // Resize the incoming new view
+    NSView* newView = [viewCon view] ;
+    NSRect frame = NSMakeRect(
+                              [newView frame].origin.x,
+                              [newView frame].origin.y,
+                              [self frame].size.width,
+                              [self frame].size.height
+                              ) ;
+    [newView setFrame:frame] ;
+
+    // Place the incoming new view.
+    [self addSubview:newView] ;
+    [self display] ;
+    
+#if 0
+    /*
+     I considered actually swapping in [viewCon view] in place of the
+     receiver, and allowing the receiver to be released and deallocced.
+     But that is too messy because not only does the tab view item need
+     to get a new view, but the outlet to this Lazy View, needed for other
+     purposes, from the window controller, would need to be rewired.
+     Before realizing that this was approach was the more problematic,
+     I solved the first problem, but not the second, by doing this…
+     */
+    if (parentTabViewItem) {
+        // parentTabViewItem is an outlet.  (More mess)
+        [parentTabViewItem setView:[viewCon view]] ;
+    }
+    else {
+        NSView* superview = [self superview] ;
+        [superview addSubview:[viewCon view]] ;
+    }
+#endif
 
     [self setIsLoaded:YES] ;
 }
 
 - (void)viewDidMoveToWindow {
+    /*SSYDBL*/ NSLog(@"%@ did move to window", self) ;
 	[super viewDidMoveToWindow] ;
 
 #if 11
-#define LAZY_LOAD_DELAY 1.0
+#define LAZY_LOAD_DELAY 0.0
 #endif
-    [self performSelector:@selector(load)
-               withObject:nil
+    [self performSelector:@selector(loadInWindow:)
+               withObject:[self window]
                afterDelay:LAZY_LOAD_DELAY] ;
 }
 
 - (void)dealloc {
+    /*SSYDBL*/ NSLog(@"%@ is dealloccing", self) ;
     [m_viewController release] ;
 
     [super dealloc] ;

SSYTabView.m

@@ -51,16 +51,6 @@ - (void)setSelectedTabIndex:(NSInteger)selectedTabIndex {
 		// overrides below.
 		m_lockOutInfiniteLoop = YES ;
 		[super selectTabViewItemAtIndex:selectedTabIndex] ;
-        /*SSYDBL*/ NSLog(@"Did select tab view item by index %ld", (long)selectedTabIndex) ;
-#if 11
-#warning Doing More
-        NSTabViewItem* item = [[self tabViewItems] objectAtIndex:selectedTabIndex] ;
-        [super selectTabViewItem:item] ;
-        /*SSYDBL*/ NSLog(@"Did select tab view item by item %@", item) ;
-        NSString* selectedIdentifier = [item identifier] ;
-        [super selectTabViewItemWithIdentifier:selectedIdentifier] ;
-        /*SSYDBL*/ NSLog(@"Did select tab view item by identifier %@", selectedIdentifier) ;
-#endif
 
 		// When the above message invokes any of the methods we have
 		// overridden below, they'll route to super too.
@@ -73,7 +63,6 @@ - (void)setSelectedTabIndex:(NSInteger)selectedTabIndex {
 
 - (void)selectTabViewItem:(NSTabViewItem *)tabViewItem {
 	if (m_lockOutInfiniteLoop) {
-        /*SSYDBL*/ NSLog(@"Super select item %@", tabViewItem) ;
 		[super selectTabViewItem:tabViewItem] ;
 	}
 	else {
@@ -86,7 +75,6 @@ - (void)selectTabViewItem:(NSTabViewItem *)tabViewItem {
 
 - (void)selectTabViewItemAtIndex:(NSInteger)index {
 	if (m_lockOutInfiniteLoop) {
-        /*SSYDBL*/ NSLog(@"Super select index %ld", (long)index) ;
 		[super selectTabViewItemAtIndex:index] ;
 	}
 	else {
@@ -96,7 +84,6 @@ - (void)selectTabViewItemAtIndex:(NSInteger)index {
 
 - (void)selectTabViewItemWithIdentifier:(id)identifier {
 	if (m_lockOutInfiniteLoop) {
-        /*SSYDBL*/ NSLog(@"Super select identifier %@", identifier) ;
 		[super selectTabViewItemWithIdentifier:identifier] ;
 	}
 	else {
@@ -116,10 +103,6 @@ - (IBAction)changeTabViewItem:(NSToolbarItem*)sender {
 }
 
 - (void)awakeFromNib {
-#if 11
-#warning More wildness
-    /*SSYDBL*/ NSLog(@"Tab View is %@", self) ;
-#endif
     // Set the initial state
 	NSTabViewItem* selectedTabViewItem = [self selectedTabViewItem] ;
 	// We have required that corresponding tab view items and toolbar items