[7u4] Request for review: 7124537 [macosx] Menu shortcuts for all menu items should be disabled if a menu itself is disabled
anthony.petrov at oracle.com
Wed Feb 29 07:52:00 PST 2012
Oh, I see. Thanks for clarifying that. Looks fine then.
On 2/29/2012 7:50 PM, Sergey Bylokhov wrote:
> 29.02.2012 19:06, Anthony Petrov wrote:
>> Hi Sergey,
>> I'm not an expert in the Menu code, but generally the fix looks good.
>> By the way, LWCToolkit.java doesn't contain any changes. Should it?
> There is a changes in formatting
>> best regards,
>> On 2/29/2012 4:50 PM, Sergey Bylokhov wrote:
>>> Does anybody has a chance to review it?
>>> 22.02.2012 22:08, Sergey Bylokhov wrote:
>>>> Hi Everyone,
>>>> Change description:
>>>> 1 CMenu.java: state of all elements for the current menu changes
>>>> now. Before the fix, menu in menubar didn't change status.
>>>> 2 CMenuItem.java: menu item can be enabled only if parent is enabled
>>>> too. Native code execute only if status was changed.
>>>> Fix was created on the java level instead of native. Please comment
>>>> if you have an objections.
>>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7124537
>>>> Webrev can be found at:
More information about the macosx-port-dev