Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327650 - No Menu Entries in Tree Popup
Summary: No Menu Entries in Tree Popup
Status: VERIFIED FIXED
Alias: None
Product: RAP
Classification: RT
Component: Workbench (show other bugs)
Version: 1.4   Edit
Hardware: All All
: P2 major (vote)
Target Milestone: 1.4 M3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-13 07:53 EDT by Markus Krüger CLA
Modified: 2010-10-15 08:00 EDT (History)
2 users (show)

See Also:

Attachments
Patch for the Demo to reproduce (105.79 KB, text/plain)
2010-10-13 07:53 EDT, Markus Krüger CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Krüger CLA 2010-10-13 07:53:29 EDT 
Created attachment 180756 [details]
Patch for the Demo to reproduce

I have a view that displays a tree in it (Actually it is a bit more complicated)
In a CVS checkout of 1.3 of the date 20.05.2010 everything works and the sample command is shown in the tree menu.
With 1.4 and CVS checkout of today the sample command is not shown, but it seems to be recognized as the menu stays open (usually empty menus pop up and close right away).

I attached a patch for the RAP demo. As out app is a bit complicated I copied a lot of classes to it, as a simple tree in view with menu contributions worked.
Comment 1 Tim Buschtoens CLA 2010-10-14 11:24:08 EDT 
Fixed in CVS HEAD.

There were two menus (in this case one on the tree, one on the expandbar), both got opend on the client, but only one reported to the server (the empty one on the expandbar). Fixed by calling stopProppagation in the menuHandler, so only the first, correct menu (the tree one) is opend.
Comment 2 Markus Krüger CLA 2010-10-15 08:00:40 EDT 
I can verify that it now works as expected.