Re: [PATCH] Re: bug in popup button

[Serg Stoyan]

Hello Michael,

> Ok, I've attached a patchset, against the latest CVS code, which
> implements these behaviors without fidgeting about with "hidden"
> menuitems. It even beeps when you click on, or activate with a key click,
> a popupbutton which has zero items.
> 
> It compiles here, (tm), and seems to work well with the popuptest that
> Benhur sent a few days ago. I am very green when it comes to reference
> counting so if someone could check that I'd appreciate it. I put comments
> in the code so hopefully y'all can follow my thinking.

  Brilliant, Michael! I've almost implemented the same, but lack of time...
  Anyway, I've committed your patch with some minor changes to CVS.

> There are some issues with the menu item cell size calculations that are
> brought to light with the PopUpTest.app, but that is a seperate topic.

  Size? I've mentioned problem with nibble position in popup button loaded
  from the .gorm, and no problem with size. And yes, that is a separate
  topic.

-- 
Serg Stoyan