Re: Last change in NSMenuView.m

[Germán Arias]

This attached patch fix the behavior of menu in window. And add code to
display transient menus in a better way. For example, if the transient
menu is displayed at right side of the screen, its submenus are
displayed to left. This avoids have submenus displayed above its
supermenu, and that not respond properly.

Also, I added code to display the transient menus with one of its bottom
corners below the mouse cursor (when transient menus are displayed near
of bottom). With this the transient menus at appicons are more
consistently, and respond correctly. Or I think so.

Try the patch, add changes, discuss the code or apply it. Regards.


On mar, 2011-01-18 at 00:05 -0600, Germán Arias wrote:
> I think:
> 
>  && ![[self menu] isTransient]
>  
> really solve the problem. But in other hand, the transient menus have
> one bug. This bug occurs when transient menu is displayed at bottom. In
> this case the transient menu moves up, to appears entirely at screen.
> When this occurs, the transient menu don't respond to the events
> correctly. You can see this problem with the transient menu at AppIcons,
> when the icons are displayed at bottom. Or in any transient menu
> displayed at bottom. For example, in an Ink document that is near at
> bottom. This bug is independently of the menu style.
> 
> 
> On lun, 2011-01-17 at 14:08 -0500, Gregory Casamento wrote:
> > I'm also not entirely certain how going into that code when the
> > [[[NSApp mainWindow] menu] attachedMenu] method returns nil helps us
> > in the first place.
> > 
> > 
> > GC
> > 
> > On Jan 17, 2011, at 1:25 PM, Germán Arias wrote:
> > 
> > > Maybe:
> > > 
> > > && ![[self menu] isTransient]
> > > 
> > > is better. But even with this, and independently of the menu style,
> > > there are some problems with the transient menu at appicon. Seems
> > > like
> > > don't track correctly the mouse. 
> > > 
> > > On lun, 2011-01-17 at 10:20 -0500, Gregory Casamento wrote:
> > > > The issue I was trying to correct was that pull down menus were
> > > > not
> > > > working with the previous code.
> > > > 
> > > > I will test the situation you describe below as well.
> > > > 
> > > > GC
> > > > 
> > > > On Monday, January 17, 2011, Germán Arias <address@hidden>
> > > > wrote:
> > > > > Hi Greg.
> > > > > 
> > > > > The addition at lines 1660 and 1661 in NSMenuView.m:
> > > > > 
> > > > > && anAttachedMenu != nil
> > > > > 
> > > > > added an odd behavior at menu in window. This is because if you
> > > > > move the
> > > > > mouse outside a submenu, the horizontal menu retrieves the mouse
> > > > > tracking. But if the mouse is outside this too, the submenu is
> > > > > closed.
> > > > > Then in next loop,there isn't an attached menu, and the code to
> > > > > handle
> > > > > the menu in window is not executed, and variable "space" never
> > > > > arrive to
> > > > > 2. Then we can't break the loop. We need add 1 to variable
> > > > > "space" even
> > > > > when there isn't an attached menu. Or I think so, I will test
> > > > > this
> > > > > tomorrow.
> > > > > 
> > > > > 
> > > > > _______________________________________________
> > > > > Gnustep-dev mailing list
> > > > > address@hidden
> > > > > http://lists.gnu.org/mailman/listinfo/gnustep-dev
> > > > > 
> > > > 
> > > 
> > > 
> > > _______________________________________________
> > > Gnustep-dev mailing list
> > > address@hidden
> > > 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)
> > 
> >

transientMenus.patch
Description: Text Data