bug-gnustep
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Menu code somewhat changed


From: Richard Frith-Macdonald
Subject: Re: [PATCH] Menu code somewhat changed
Date: Mon, 24 Feb 2003 08:13:01 +0000


On Sunday, February 23, 2003, at 08:02  pm, Willem Rein Oudshoorn wrote:

The following patch changes slightly the mechanism
of the NSMenu (View) code.

Changed behaviour
-----------------

* Remove highlighting explicitly after executing action.
  (I guess the restore highlighting code was put there for
   a reason.  But I could not think of any.)

* Remove the temporary menu before executing the
  action.

I this this out and found a few problems/bugs introduced
by the patch....

When you select a menu item which brings up a submenu,
the menu item should remain highlighted until the submenu
is either closed or torn off, so you can see at a glance which
menu item you should click to close the submenu.

After your patch, the highlighting was missing ... and worse,
clicking on the menu item again failed to close the submenu!

Other than that, the patch looks good, and if you can fix the
highlighting errors so that submenus operate correctly
again, I'll commit it.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]