Re: Table menu patch for GtkMenu

[Matthias Clasen <maclas gmx de>]

Am Sam, 2003-07-12 um 20.57 schrieb Kristian Rietveld:
> On Sat, 2003-07-12 at 20:52, Matthias Clasen wrote:
> > Am Sam, 2003-07-12 um 17.22 schrieb Kristian Rietveld:
> > > Hello,
> > > 
> > > Before I can land the new combo box in gtk+ HEAD, I need to get my happy
> > > fun table menu patch for GtkMenu approved. So I attached this patch. A
> > > good chunk of the code is based on the GtkTable size
> > > allocation/requisition code. I guess it's a pretty sane patch, but maybe
> > > some won't like the proposed API.
> > > 
> > > 
> > > thanks,
> > > 
> > > 
> > > 	-Kris
> > 
> > No docs...could you explain the difference between _attach() and
> > _set_attach() again ? They look pretty similar at first glance.
> 
> I will add docs before I commit. _attach() was for adding new childs,
> and _set_attach() for modifying current childs (but it also handles new
> childs I see... (long time ago)). I think I should just drop _attach()
> and move _set_attach() to _attach().
> 
> Sounds good?

Yes