Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-11-01 Thread Fred Kiefer
Hi Doug, sorry for replying so late. I was away for a week on holiday (and well needed holidays as I have to say). My original suggestion was to keep everything within the WinUX theme. My thought was that the whole logic that you implemented into NSPopupButtonCell (or was it NSMenuView?) of remov

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-10-27 Thread Doug Simons
Solved! Okay, after much thought and carefully talking through this with Greg and my colleague Jonathan, we've come up with what seems to be a good solution. This involves adding a new theme method which allows us to inquire whether the current theme handles event processing for a popUp menu. I

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-10-27 Thread Doug Simons
Thanks for the suggestion, Kai. But the problem is that we'd like to not change existing GNUstep code in awkward ways. The code uses the standard nextEventMatchingMask:... method to get the next event, which inevitably calls some Windows function that allows the callbacks to come in. Fortunatel

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-10-26 Thread Doug Simons
Hello Fred, Greg, and anyone else who cares to look at this problem with Windows events. Unfortunately, without the former fix in place the pulldown menus are not working correctly on Windows. Here's the situation: Our action method gets called twice on Windows when selecting an item from a pu

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-10-19 Thread Fred Kiefer
Hi Doug, sorry the revert is already committed, but if there is a way to help you I am willing to jump in. In my first mail I suggested to add the popup test into the processCommand: method of the tehme. I still think this should work, but it may not be the best way to resolve the issue. Greg, who

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-10-19 Thread Doug Simons
Hi Fred, I apologize for letting this slide and not responding before. The changes I made definitely resolved some specific problems that we were seeing, but perhaps not solved in the best possible way. I certainly understand your point about addressing any Windows-specific issues in the backen

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-10-17 Thread Fred Kiefer
I never got a reply on this mail. I will now undo this dubious change. If it was really required for the WinUX theme I hope that somebody will add a corresponding change into that theme. I really would have preferred to have some discussion on this subject. Fred Am 12.09.2010 20:49, schrieb Fred

Re: [Gnustep-cvs] r31213 - in /libs/gui/trunk: ChangeLog Source/NSMenuView.m Source/NSPopUpButtonCell.m

2010-09-12 Thread Fred Kiefer
Am 31.08.2010 01:02, schrieb Doug Simons: > Author: dpsimons > Date: Tue Aug 31 01:02:21 2010 > New Revision: 31213 > > URL: http://svn.gna.org/viewcvs/gnustep?rev=31213&view=rev > Log: > fix problem of pulldown action not being called for correct cell, and being > called twice on Windows > > Mo