KPushButton::setDelayedMenu

[Allen Winter]

On Wednesday 18 July 2007 7:59:40 am Thomas McGuire wrote:
> Hi,
> 
> On Saturday 07 July 2007, Dominik Haumann wrote:
> > KPushButton is pretty much broken with regard to the delayed menu
> > capabilities. As QPushButton::setMenu is not virtual there is an additional
> > function KPushButton::setDelayedMenu().
> >
> > When you hold down the button long enough a timer ticks which then adds the
> > menu by using QPushButton::setMenu(delayed_menu).
> >
> > That's also the reason why KPushButton does not show the "down arrow" on
> > the right side of the button even if you set a delayed menu. The arrow is
> > shown when you hold down the button long enough. That means that there is
> > no indication that the button has a delayed menu at all.
> I tried your patch because I had hopes that it would fix the arrow that are 
> not shown for toolbar buttons which have a menu. Examples of these are the 
> file selection dialog, which has a configure button, or KMail's check mail, 
> reply and forward buttons.
> 
> However, they are still the same, so they either don't use KPushButton or 
> something went wrong with the patch.
> 
> It is important that the arrow is shown in this case, otherwise the user will 
> never discover the functionality.
> 
> > I'm sorry for not providing a proper .patch file, but I've locally change
> > the file in other places as well ;)
> Anyway, I've made a patch out of it. See attached.
> 
> Hopefully someone will have a look at this patch and at the toolbar arrow 
> issue.
> 
Was this issue ever resolved?
I don't see anything that resembles Thomas' patch in kpushbutton.cpp