Daniel Åkesson wrote:

my($x, $y) = Win32::GUI::GetCursorPos();

$mw_win32->TrackPopupMenu($systray_menu->{SystrayMenu}, $x, $y);

My most humble apologies. I've broken TrackPopupMenu() in a piece of logic that I spent so long looking at that I was *sure* I had it right. The fix is trivial, and I've put it into CVS. If you want to build your own immediately I can send you a patch. If not try one of these:

$mw_win32->TrackPopupMenu($systray_menu->{SystrayMenu});

will, as of V1.02, do the GetCursorPos() for you.


If you need backwards compatibility with earlier versions of Win32::GUI, then

$mw_win32->TrackPopupMenu($systray_menu->{SystrayMenu}, $x, $y, TPM_LEFTALIGN | 
TPM_TOPALIGN | TPM_LEFTBUTTON );

should work in all cases.  (I've broken the 3-parameter call variant)

I've raised a tracker: 1241830, so that there is visibility of the problem.  
Does anyone think this is serious enough (as it breaks backwards compatibility) 
to warrant thinking about another release?

Regards,
Rob.






Reply via email to