RE: [Plib-devel] Re: [BUG] puComboBox broken since recent puInput Base commit?

[Fay J. F Dr C. U. AFSEO/SK <joh...@eg...>]

Gentlemen,

	To fix the "puaComboBox" widget, please add the following methods to
the "puInput" widget definition:

<begin quote>
  virtual void setValue ( puValue *pv )
  {
    puValue::setValue ( pv ) ;
    delete displayed_text ;  displayed_text = NULL ;
  }

  virtual void setValue ( int   i )
  {
    puValue::setValue ( i ) ;
    delete displayed_text ;  displayed_text = NULL ;
  }

  virtual void setValue ( float f )
  {
    puValue::setValue ( f ) ;
    delete displayed_text ;  displayed_text = NULL ;
  }

  virtual void setValue ( const char *s )
  {
    puValue::setValue ( s ) ;
    delete displayed_text ;  displayed_text = NULL ;
  }

  virtual void setValue ( bool b )
  {
    puValue::setValue ( b ) ;
    delete displayed_text ;  displayed_text = NULL ;
  }
<end quote>

This fixes Melchior's and Bram's test case on my Windows box.  If somebody
would please put it into CVS I would appreciate it.

John F. Fay
Technical Fellow, Jacobs/Sverdrup TEAS Group
850-729-6330
joh...@eg...
 
-----Original Message-----
From: pli...@li...
[mailto:pli...@li...] On Behalf Of Melchior FRANZ
Sent: Tuesday, March 28, 2006 10:52 AM
To: pli...@li...
Subject: [Plib-devel] Re: [BUG] puComboBox broken since recent puInputBase
commit?

* Bram Stolk -- Tuesday 28 March 2006 18:19:
> Melchior FRANZ wrote:
> > It looks like the recent puInput{,Base} commit broke comboboxes?
> > Can someone check (and fix :-)?  Selecting a puPopupMenu entry 
> > doesn't update the puInput any more.

> Do you have a test sample?

Attached. It's derived from simple.cxx. With a plib from before the
puInputBase changes (about two or three weeks back), the input field is
updated if you select an entry from the popup. With HEAD it isn't.



> I never understood "widget_list" from examples/src/pui If I start 
> that, it opens a gazzillion windows, none of which are responsive.

I've well understood that it's meant to be as useless as it is. I really
wonder why there's no application that demonstrates all available widgets.

m.