Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 457886

Summary: Orion File/Edit/View menubars don't behave like real menubars
Product: [ECD] Orion Reporter: Mark Macdonald <mamacdon>
Component: ClientAssignee: Eric Moffatt <emoffatt>
Status: CLOSED WONTFIX QA Contact:
Severity: minor    
Priority: P3 CC: elijahe
Version: 7.0   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 496678    
Bug Blocks:    

Description Mark Macdonald CLA 2015-01-19 15:59:44 EST 
I noticed 2 ways in which the menubars shown on the editor page (File, Edit, View, etc) do not behave like menubars in other windowing systems**:

 I) Orion's menubars open on a "click" event, i.e. you have to release the mouse button in order to see the menu contents.

 Other windowing systems show the menu on mousedown. This feels snappier.

 2) You cannot quickly scan through several menus by opening one and then hovering over the others with the mouse cursor -- you must click each menu individually.

These are minor issues, but they contribute to a perception of slowness/clunkiness in the UI.

** Tested Windows, OS X, and Google Docs
Comment 1 Eric Moffatt CLA 2016-07-27 10:57:25 EDT 
OK, so the first step here is to gather up all the menu processing / rendering into a simpler structure. We need to replace the current hard-coded calls to 'addCommandGroup' and 'registerCommandContribution' with fixed structures. Ultimately there should be *one* structure that completely defines the 'main menu' and augment the '_render' to produce everything including the menu 'bar'...
Comment 2 Eric Moffatt CLA 2016-08-04 11:41:33 EDT 
Ive captured the initial work into a branch "MenuStructs". This is phase 1...

It tackles the Editor page and currently has replacements for the main menu bar's dropdowns (File / Edit / View / Tools) as well as both the nav and editor context menus.

This is currently under a darklaunch with the test being:

var useStructs = "true" === localStorage.getItem("useMenuStruct")
Comment 3 Eric Moffatt CLA 2016-08-04 12:02:09 EDT 
We've decided to capture the rest of this work on a more specific defect; bug 496678
Comment 4 Michael Rennie CLA 2017-01-10 15:43:57 EST 
Closing as part of a mass clean up of inactive bugs. Please reopen if this problem still occurs or is relevant to you. For more details see:

https://dev.eclipse.org/mhonarc/lists/orion-dev/msg04002.html