Re: ComboBox in TableView no longer works

[Fred Kiefer]

Quentin Mathé wrote:
> Le 24 mai 05 à 20:08, Andreas Höschler a écrit :
>
> > about a year ago combo boxes in a tableView worked (thanks to  Fred). 
> > After a recent CVS update this functionality is gone again.  I have 
> > attached a very simple test program to demonstrate this. On  Mac OS X 
> > the combo pops open if the button of the combo is clicked.  On GNUstep 
> > this no longer works. Is anybody aware of the changes  that were made 
> > during the last months that could cause this loss of  functionality?
>
> I don't remember precisely the whole story now. But IIRC I would say  
> combo boxes never worked in table view, because table view had and  
> still has a broken cell support. However in June 2004 Fred committed  a 
> patch which improved combo box and a bit cell support in table  view, 
> then combo boxes inside table view seemed to work in some  special 
> cases. Later I discovered standalone combo boxes weren't  usable 
> anymore, this issue was the result of Fred's patch. When I  tracked the 
> bug origin, I came to this conclusion : to have both  standalone and 
> embedded combo boxes working would involve rewriting  table view cell 
> support (very complicated task). That's why I decided  it was better to 
> support first standalone combo boxes and not  embedded until I or 
> somebody may find some time to fix cell support  in table view.

Hi Quentin,

surely everybody agrees that the cell support in table view is broken 
and has been since it was implemented. But as far as I remember basic 
combo box behaviour was working stand alone and in table views. What 
broke it was another change in table view, done for totally different 
reason. Currently Matt Rice is proposing patches to correct the 
behaviour of the table view, which look very promissing. Perhaps you 
could join in on testing and reviewing them. I am sure we get that 
sorted out for the next GNUstep release.
The bad news is that similar clean-ups are also needed for NSMatrix and 
all the other classes handling more than one cell. But we will get there.

Fred