Initial implementation of NSBrowser and NSBrowserCell.

Numerous bug fixes.


git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/gui/trunk@2596 72102866-910b-0410-8b05-ffd578937521

Source/GNUmakefile

@@ -76,6 +76,7 @@ NSMenuItem.m \
 NSOpenPanel.m \
 NSPageLayout.m \
 NSPanel.m \
+NSPasteboard.m \
 NSPrinter.m \
 NSPrintInfo.m \
 NSPrintOperation.m \
@@ -102,7 +103,6 @@ TrackingRectangle.m \
 PSMatrix.m \
 tiff.m \
 externs.m
-# NSPasteboard.m
 
 libgnustep-gui_HEADER_FILES_DIR = ../Headers
 libgnustep-gui_HEADER_FILES_INSTALL_DIR = /gnustep/gui

Source/NSBrowserCell.m

@@ -3,7 +3,7 @@
 
    Cell class for the NSBrowser
 
-   Copyright (C) 1996 Free Software Foundation, Inc.
+   Copyright (C) 1996, 1997 Free Software Foundation, Inc.
 
    Author:  Scott Christley <scottc@net-community.com>
    Date: 1996
@@ -28,7 +28,42 @@
 
 #include <gnustep/gui/config.h>
 #include <AppKit/NSBrowserCell.h>
+#include <AppKit/NSTextFieldCell.h>
+#include <AppKit/NSImage.h>
 
+// Class variables
+static NSImage *branch_image;
+static NSImage *highlight_image;
+
+// Private methods
+@interface NSBrowserCell (Private)
+- (void)setBranchImageCell:aCell;
+- (void)setHighlightBranchImageCell:aCell;
+- (void)setTextFieldCell:aCell;
+@end
+
+@implementation NSBrowserCell (Private)
+
+- (void)setBranchImageCell:aCell
+{
+  _branchImage = aCell;
+}
+
+- (void)setHighlightBranchImageCell:aCell
+{
+  _highlightBranchImage = aCell;
+}
+
+- (void)setTextFieldCell:aCell
+{
+  _browserText = aCell;
+}
+
+@end
+
+//
+// NSBrowserCell implementation
+//
 @implementation NSBrowserCell
 
 //
@@ -40,6 +75,10 @@
     {
       // Initial version
       [self setVersion:1];
+
+      // The default images
+      branch_image = [NSImage imageNamed: @"common_ArrowRight"];
+      highlight_image = [NSImage imageNamed: @"common_ArrowRightH"];
     }
 }
 
@@ -48,59 +87,177 @@
 //
 + (NSImage *)branchImage
 {
-  return nil;
+  return branch_image;
 }
 
 + (NSImage *)highlightedBranchImage
 {
-  return nil;
+  return highlight_image;
 }
 
 //
 // Instance methods
 //
 
+- init
+{
+  return [self initTextCell: @"aTitle"];
+}
+
+- initTextCell:(NSString *)aString
+{
+  [super initTextCell: aString];
+
+  // Our image cells
+  _branchImage = [[NSCell alloc] initImageCell: [NSBrowserCell branchImage]];
+  _highlightBranchImage = [[NSCell alloc] initImageCell:
+			        [NSBrowserCell highlightedBranchImage]];
+
+  // Our text cell
+  _browserText = [[NSTextFieldCell alloc] initTextCell: aString];
+  [_browserText setEditable: NO];
+  [_browserText setBordered: NO];
+  [_browserText setDrawsBackground: YES];
+
+  _alternateImage = nil;
+  _isLeaf = NO;
+  _isLoaded = NO;
+
+  return self;
+}
+
+- (void)dealloc
+{
+  [_branchImage release];
+  [_highlightBranchImage release];
+  [_alternateImage release];
+  [_browserText release];
+
+  [super dealloc];
+}
+
+- (id)copyWithZone:(NSZone*)zone
+{
+  NSBrowserCell* c = [super copyWithZone:zone];
+
+  // Copy the image cells
+  [c setBranchImageCell: [_branchImage copy]];
+  [c setHighlightBranchImageCell: [_branchImage copy]];
+  [c setAlternateImage: _alternateImage];
+
+  // Copy the text cell
+  [c setTextFieldCell: [_browserText copy]];
+
+  [c setLeaf: _isLeaf];
+  [c setLoaded: NO];
+
+  return c;
+}
+
 //
 // Accessing Graphic Attributes 
 //
 - (NSImage *)alternateImage
 {
-  return nil;
+  return _alternateImage;
 }
 
 - (void)setAlternateImage:(NSImage *)anImage
-{}
+{
+  [anImage retain];
+  [_alternateImage release];
+  _alternateImage = anImage;
+
+  // Set the image in our highlight cell
+  if (_alternateImage)
+    [_highlightBranchImage setImage: _alternateImage];
+  else
+    [_highlightBranchImage setImage: [NSBrowserCell highlightedBranchImage]];
+}
 
 //
 // Placing in the Browser Hierarchy 
 //
 - (BOOL)isLeaf
 {
-  return NO;
+  return _isLeaf;
 }
 
 - (void)setLeaf:(BOOL)flag
-{}
+{
+  _isLeaf = flag;
+}
 
 //
 // Determining Loaded Status 
 //
 - (BOOL)isLoaded
 {
-  return NO;
+  return _isLoaded;
 }
 
 - (void)setLoaded:(BOOL)flag
-{}
+{
+  _isLoaded = flag;
+}
 
 //
 // Setting State 
 //
 - (void)reset
-{}
+{
+  cell_highlighted = NO;
+  cell_state = NO;
+}
 
 - (void)set
-{}
+{
+  cell_highlighted = YES;
+  cell_state = YES;
+}
+
+//
+// Setting the NSCell's Value 
+//
+- (double)doubleValue
+{
+  return [_browserText doubleValue];
+}
+
+- (float)floatValue;
+{
+  return [_browserText floatValue];
+}
+
+- (int)intValue
+{
+  return [_browserText intValue];
+}
+
+- (NSString *)stringValue
+{
+  return [_browserText stringValue];
+}
+
+- (void)setDoubleValue:(double)aDouble
+{
+  [_browserText setDoubleValue: aDouble];
+}
+
+- (void)setFloatValue:(float)aFloat
+{
+  [_browserText setFloatValue: aFloat];
+}
+
+- (void)setIntValue:(int)anInt
+{
+  [_browserText setIntValue: anInt];
+}
+
+- (void)setStringValue:(NSString *)aString
+{
+  [_browserText setStringValue: aString];
+}
 
 //
 // NSCoding protocol
@@ -108,12 +265,24 @@
 - (void)encodeWithCoder:aCoder
 {
   [super encodeWithCoder:aCoder];
+
+  [aCoder encodeObject: _browserText];
+  [aCoder encodeObject: _branchImage];
+  [aCoder encodeObject: _highlightBranchImage];
+  [aCoder encodeValueOfObjCType: @encode(BOOL) at: &_isLeaf];
+  [aCoder encodeValueOfObjCType: @encode(BOOL) at: &_isLoaded];
 }
 
 - initWithCoder:aDecoder
 {
   [super initWithCoder:aDecoder];
 
+  _browserText = [aDecoder decodeObject];
+  _branchImage = [aDecoder decodeObject];
+  _highlightBranchImage = [aDecoder decodeObject];
+  [aDecoder decodeValueOfObjCType: @encode(BOOL) at: &_isLeaf];
+  [aDecoder decodeValueOfObjCType: @encode(BOOL) at: &_isLoaded];
+
   return self;
 }
 

Source/NSCell.m

@@ -98,6 +98,8 @@
 
 - (id)initImageCell:(NSImage *)anImage
 {
+  [super init];
+
   [self _init];
 
   // Not an image class --then forget it
@@ -113,6 +115,8 @@
 
 - (id)initTextCell:(NSString *)aString
 {
+  [super init];
+
   [self _init];
 
   cell_font = [[NSFont userFontOfSize:12] retain];

Source/NSClipView.m

@@ -59,7 +59,7 @@
 {
   [super init];
   [self setAutoresizesSubviews:YES];
-  [self setBackgroundColor:[[self window] backgroundColor]];
+  [self setBackgroundColor:[NSColor lightGrayColor]];
   return self;
 }
 

Source/NSMatrix.m

@@ -132,7 +132,7 @@ static void growMatrix (tMatrix m, int numRows, int numCols)
     /* Grow the existing rows to numCols */
     for (i = 0; i < numRows; i++) {
       m->matrix[i] = realloc (m->matrix[i], numCols * sizeof (BOOL));
-      for (j = m->allocatedCols; j < numCols; j++)
+      for (j = m->allocatedCols - 1; j < numCols; j++)
 	m->matrix[i][j] = NO;
     }
     m->allocatedCols = numCols;
@@ -422,9 +422,9 @@ static int mouseDownFlags = 0;
   int i;
 
   /* Grow the matrix if necessary */
-  if (column > numCols)
+  if (column >= numCols)
     [self renewRows:(numRows == 0 ? 1 : numRows) columns:column];
-  else if (numRows == 0) {
+  if (numRows == 0) {
     numRows = 1;
     [cells addObject:[NSMutableArray array]];
   }
@@ -443,9 +443,9 @@ static int mouseDownFlags = 0;
   int i;
 
   /* Grow the matrix if necessary */
-  if (column > numCols)
+  if (column >= numCols)
     [self renewRows:(numRows == 0 ? 1 : numRows) columns:column];
-  else if (numRows == 0) {
+  if (numRows == 0) {
     numRows = 1;
     [cells addObject:[NSMutableArray array]];
   }
@@ -465,9 +465,9 @@ static int mouseDownFlags = 0;
   int i;
 
   /* Grow the matrix if necessary */
-  if (row > numRows)
+  if (row >= numRows)
     [self renewRows:row columns:(numCols == 0 ? 1 : numCols)];
-  else if (numCols == 0)
+  if (numCols == 0)
     numCols = 1;
 
   [cells insertObject:[NSMutableArray arrayWithCapacity:numCols] atIndex:row];
@@ -485,9 +485,9 @@ static int mouseDownFlags = 0;
 - (void)insertRow:(int)row withCells:(NSArray*)cellArray
 {
   /* Grow the matrix if necessary */
-  if (row > numRows)
+  if (row >= numRows)
     [self renewRows:row columns:(numCols == 0 ? 1 : numCols)];
-  else if (numCols == 0)
+  if (numCols == 0)
     numCols = 1;
 
   [cells insertObject:[cellArray subarrayWithRange:NSMakeRange(0, numCols)]

Source/NSTextFieldCell.m

@@ -99,6 +99,7 @@
 
   [c setBackgroundColor: background_color];
   [c setTextColor: text_color];
+  [c setDrawsBackground: draw_background];
 
   return c;
 }

Source/NSView.m

@@ -883,18 +883,30 @@ static NSRecursiveLock *gnustep_gui_nsview_lock = nil;
 
 - (void)_removeSubviewFromViewsThatNeedDisplay:(NSView*)view
 {
+  // If no subviews need to be displayed then
+  // then view must not be among them
+  if (!_subviewsThatNeedDisplay)
+    return;
+
   /* Remove view from the list of subviews that need display */
   if (_subviewsThatNeedDisplay == view)
-    _subviewsThatNeedDisplay = view->_nextSiblingSubviewThatNeedsDisplay;
+    {
+      _subviewsThatNeedDisplay = view->_nextSiblingSubviewThatNeedsDisplay;
+      [view _recursivelyResetNeedsDisplayInAllViews];
+    }
   else {
     NSView* currentView;
 
     for (currentView = _subviewsThatNeedDisplay;
-	 currentView->_nextSiblingSubviewThatNeedsDisplay;
+	 currentView && currentView->_nextSiblingSubviewThatNeedsDisplay;
 	 currentView = currentView->_nextSiblingSubviewThatNeedsDisplay)
       if (currentView->_nextSiblingSubviewThatNeedsDisplay == view)
-	currentView->_nextSiblingSubviewThatNeedsDisplay
+	{
+	  currentView->_nextSiblingSubviewThatNeedsDisplay
 	    = view->_nextSiblingSubviewThatNeedsDisplay;
+	  [view _recursivelyResetNeedsDisplayInAllViews];
+	  break;
+	}
   }
 }
 
@@ -934,6 +946,7 @@ static NSRecursiveLock *gnustep_gui_nsview_lock = nil;
     currentView = nextView;
   }
   _subviewsThatNeedDisplay = NULL;
+  _nextSiblingSubviewThatNeedsDisplay = NULL;
 }
 
 - (void)_displayNeededViews