Shouldn't we go ahead and apply this patch? It looks like it does the trick and it also does things were was suggested earlier in NSMenuView.m.
GC On Fri, Jan 14, 2011 at 2:02 PM, Germán Arias <[email protected]> wrote: > On vie, 2011-01-14 at 16:11 +0100, Fred Kiefer wrote: >> Hi German, >> >> your latest patch looks very promising to me. What I didn't understand >> is the line >> + else >> + { >> + [[self keyWindow] makeMainWindow]; >> + } >> >> When there is already a main Window, shouldn't we just keep that? >> I will have a closer look at the whole patch over the weekend. >> >> Thank you for your contribution >> Fred >> > > I thought that this was necessary for the apps that were not prepared to > have the menu in window. But it was a bad argument, the problem is that > we need to save the main window in the metod -hide:, too. Otherwise the > windows are order out and we lose the main window, and method > -deactivate: don't save the main window, because there isn't. This is > the same that NSApp do for key window, save the key window at methods > -hide and -deactivate. Attached the new patch. > > Regards. > > > _______________________________________________ > Gnustep-dev mailing list > [email protected] > http://lists.gnu.org/mailman/listinfo/gnustep-dev > > -- Gregory Casamento - GNUstep Lead/Principal Consultant, OLC, Inc. yahoo/skype: greg_casamento, aol: gjcasa (240)274-9630 (Cell) _______________________________________________ Gnustep-dev mailing list [email protected] http://lists.gnu.org/mailman/listinfo/gnustep-dev
