Changeset 433 for trunk/ICeCoffEE/ICeCoffEE/ICeCoffEEMenuOutlineView.m

Timestamp:

03/03/08 09:25:06 (16 years ago)

Author:

Nicholas Riley

Message:

Undocumented method / hack fallback to determine whether we should draw the highlighted row as active.

File:

• trunk/ICeCoffEE/ICeCoffEE/ICeCoffEEMenuOutlineView.m (modified) (3 diffs)

trunk/ICeCoffEE/ICeCoffEE/ICeCoffEEMenuOutlineView.m

@@ -10 +10 @@
 #import <Carbon/Carbon.h>
 
+// Tiger or later, indicates whether highlighted row should draw as active
+@interface NSWindow (IC_KeyboardFocus)
+- (BOOL)_isKeyWindowIgnoringFocus;
+@end
+
+BOOL ICCF_ViewHasKeyboardFocus(NSView *view) {
+    NSWindow *window = [view window];
+    if ([window firstResponder] != view)
+        return NO;
+    
+    if ([window respondsToSelector: @selector(_isKeyWindowIgnoringFocus)])
+        return [window _isKeyWindowIgnoringFocus];
+    
+    // XXX is there a documented way to do this?
+    // see <http://lists.apple.com/archives/cocoa-dev/2008/Mar/msg00036.html>
+    
+    // test cases:
+    // - About System Preferences...
+    // - focused character palette
+    // - Help menu search (Shortcut)
+    // - menu extras
+    NSWindow *keyWindow = [NSApp keyWindow];
+    if (window != keyWindow && (keyWindow == nil || [keyWindow isKindOfClass: [NSPanel class]]))
+        return NO; // exclude NSCarbonMenuWindow
+    
+    return YES;
+}
+
 @implementation ICeCoffEEMenuOutlineView
 
@@ -18 +46 @@
 }
 
-// maintains main window (e.g., window to which sheet is attached) for use by shouldUseActiveHighlight
-- (void)viewDidMoveToWindow;
-{
-    mainWindow = [NSApp mainWindow];
-    [super viewDidMoveToWindow];
-}
-
 - (BOOL)shouldUseActiveHighlight;
 {
     return NO;
-    // test cases: - XXX there should be a better way to do this
-    // see <http://lists.apple.com/archives/cocoa-dev/2008/Mar/msg00036.html>
-    // - About System Preferences...
-    // - focused character palette
-    // - Help menu search (Shortcut)
-    // - menu extras
-    NSWindow *keyWindow = [NSApp keyWindow], *thisWindow = [self window];
-    if (keyWindow != thisWindow && [keyWindow isKindOfClass: [NSPanel class]])
-        return NO; // exclude NSCarbonMenuWindow
-    return [thisWindow firstResponder] == self && [NSApp mainWindow] == mainWindow;
+    return [[self window] _isKeyWindowIgnoringFocus];
 }
 
@@ -46 +58 @@
 
     HIThemeMenuItemDrawInfo drawInfo = {0, kThemeMenuItemHierBackground | kThemeMenuItemPopUpBackground, kThemeMenuSelected};
-    NSLog(@"firstResponder %@ key %@ isKey %d main %@", [[self window] firstResponder], [NSApp keyWindow], [[self window] isKeyWindow], [NSApp mainWindow]);
-    if (![self shouldUseActiveHighlight]) {
-        instrumentObjcMessageSends(YES);
+    if (!ICCF_ViewHasKeyboardFocus(self)) {
         [super highlightSelectionInClipRect: clipRect];
-        instrumentObjcMessageSends(NO);
         return;
     }