Rewritten combo box implementation

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

ChangeLog

@@ -1,3 +1,17 @@
+2004-05-29  Quentin Mathe <qmathe@lub-internet.fr>
+        
+	* Source/NSComboBox.m:
+	* Headers/AppKit/NSComboBox.h:
+	* Source/NSComboBoxCell.m:
+	* Headers/AppKit/NSComboBoxCell.h:
+	* Images/GNUmakefile:
+	* Images/common_ComboBoxEllipsis.tiff:
+	New rewritten combo box implementation (which can be called completion
+	box now).
+	Everything should work except the completion possibility which needs some
+	layout manager and text view changes (patches to be commited by Alexander 
+	Malmberg).
+
 2004-05-29  Quentin Mathe <qmathe@lub-internet.fr>
 
 	* Source/NSControl.m: Reverted to the previous version (1.61) because 

Headers/AppKit/NSComboBox.h

@@ -27,9 +27,11 @@
 #ifndef _GNUstep_H_NSComboBox
 #define _GNUstep_H_NSComboBox
 
-#include <Foundation/NSNotification.h>
 #include <AppKit/NSTextField.h>
 
+@class NSString;
+@class NSNotification;
+
 @interface NSComboBox : NSTextField
 {
 }
@@ -82,7 +84,6 @@
 - (void) setCompletes: (BOOL)completes;
 - (BOOL) completes;
 #endif
-
 @end
 
 @interface NSObject (NSComboBoxDataSource)

Headers/AppKit/NSComboBoxCell.h

@@ -27,9 +27,13 @@
 #ifndef _GNUstep_H_NSComboBoxCell
 #define _GNUstep_H_NSComboBoxCell
 
+#include <Foundation/NSGeometry.h>
 #include <AppKit/NSTextFieldCell.h>
 
-@class NSButtonCell, NSScrollView;
+@class NSButtonCell;
+@class NSMutableArray;
+@class NSArray;
+@class NSString;
 
 @interface NSComboBoxCell : NSTextFieldCell
 {
@@ -43,6 +47,8 @@
    NSSize		_intercellSpacing;
    float		_itemHeight;
    int                  _selectedItem;
+   NSRect               _lastValidFrame;
+   NSRange		_prevSelectedRange;
   
 @private
    id		        _popup;

Images/GNUmakefile

@@ -101,6 +101,7 @@ GNUstep.tiff \
 GNUstepMenuImage.tiff \
 nsmapping.strings \
 NSComboArrow.tiff \
+common_ComboBoxEllipsis.tiff \
 common_RightTabStop.tiff \
 common_CenterTabStop.tiff \
 common_LeftTabStop.tiff \

Source/NSComboBox.m

@@ -25,15 +25,9 @@
 
 #include <Foundation/NSNotification.h>
 #include <Foundation/NSString.h>
-#include "AppKit/NSApplication.h"
 #include "AppKit/NSComboBox.h"
 #include "AppKit/NSComboBoxCell.h"
 #include "AppKit/NSEvent.h"
-#include "AppKit/NSWindow.h"
-
-@interface NSObject(MouseUpping)
-- (NSEvent *)_mouseUpEvent;
-@end
 
 /*
  * Class variables
@@ -47,7 +41,7 @@ static NSNotificationCenter *nc;
 
 + (void) initialize
 {
-  if (self == [NSComboBox class])
+  if (self == [NSComboBox class]) // Is such test case really needed ?
     {
        [self setVersion: 1];
        comboBoxCellClass = [NSComboBoxCell class];
@@ -254,23 +248,15 @@ static NSNotificationCenter *nc;
 // Overridden
 - (void) mouseDown: (NSEvent*)theEvent
 {
-  NSEvent *cEvent;
+  BOOL clicked; 
+  // clicked is set to the value YES when the click occurs in the text cell
+  // and to the value NO when it occurs in the button cell
   
-  [_cell trackMouse: theEvent inRect: [self bounds]
+  clicked = [_cell trackMouse: theEvent inRect: [self bounds] 
     ofView: self untilMouseUp: YES];
-  if ([_cell respondsToSelector: @selector(_mouseUpEvent)])
+  
-    cEvent = [_cell _mouseUpEvent];
+  if (clicked)
-  else
-    cEvent = nil;
-  if ([_cell isSelectable])
-    {
-      if (!cEvent)
-	cEvent = [NSApp currentEvent];
-      if ([cEvent type] == NSLeftMouseUp &&
-	  ([cEvent windowNumber] == [[self window] windowNumber]))
-	[NSApp postEvent: cEvent atStart: NO];
     [super mouseDown: theEvent];
 }
-}
 
 @end