Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345494 - Add support for missing MacOS UI commands from org.eclipse.ui.ide
Summary: Add support for missing MacOS UI commands from org.eclipse.ui.ide
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 4.1 RC2   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-11 16:00 EDT by Brian de Alwis CLA
Modified: 2011-05-18 14:52 EDT (History)
3 users (show)

See Also:
pwebster: review+


Attachments
patch to org.eclipse.e4.ui.workbench.renderers.swt.cocoa to add support for missing IDE commands (18.23 KB, patch)
2011-05-11 16:00 EDT, Brian de Alwis CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brian de Alwis CLA 2011-05-11 16:00:36 EDT
Created attachment 195420 [details]
patch to org.eclipse.e4.ui.workbench.renderers.swt.cocoa to add support for missing IDE commands

org.eclipse.ui.ide introduced references to 3 commands in the "Window" menu expected from MacOS X apps.  Implementations for these commands were put in org.eclipse.ui.cocoa.  The Eclipse 4 org.eclipse.e4.ui.workbench.renderers.swt.cocoa doesn't provide equivalent implementations, leading to messages on the console like the following:

   !ENTRY org.eclipse.ui 4 4 2011-05-10 20:14:24.880
   !MESSAGE Unable to create menu item "null", command
   "org.eclipse.ui.cocoa.minimizeWindow" not defined

   !ENTRY org.eclipse.ui 4 4 2011-05-10 20:14:24.881
   !MESSAGE Unable to create menu item "null", command
   "org.eclipse.ui.cocoa.zoomWindow" not defined

   !ENTRY org.eclipse.ui 4 4 2011-05-10 20:14:24.881
   !MESSAGE Unable to create menu item "null", command
   "org.eclipse.ui.cocoa.arrangeWindowsInFront" not defined

These warnings are a bit confusing and can be misleading (eg bug 345350).

The attached patch ports the E3.x implementations for these commands.  To simplify the addition, I did some minor rearrangement of the existing code.  There is no new API provided.
Comment 1 Paul Webster CLA 2011-05-16 18:49:57 EDT
Brian, I'm fine with this.  Please commit it and update the bug at your earliest convenience.

PW
Comment 2 Brian de Alwis CLA 2011-05-18 14:52:10 EDT
Fix committed to HEAD.