Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325129 - Move file menu is not enabled.
Summary: Move file menu is not enabled.
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P1 normal (vote)
Target Milestone: ---   Edit
Assignee: Bozier jerome CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard: adopter
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-13 12:26 EDT by Paul Slauenwhite CLA
Modified: 2016-05-05 10:30 EDT (History)
0 users

See Also:
paulslau: review+

Attachments
patch (2.73 KB, patch)
2010-11-30 04:30 EST, Bozier jerome CLA 		no flags Details | Diff
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 Paul Slauenwhite CLA 2010-09-13 12:26:27 EDT 
Move file menu is not enabled.

When a test asset is selected in the Test Navigator, the Move context menu is enabled, but not the Move file menu.
Comment 1 Paul Slauenwhite CLA 2010-09-13 12:27:50 EDT 
Requires a sizing.

See org.eclipse.hyades.test.ui.internal.navigator.action.MoveAction for the Move action.
Comment 2 Bozier jerome CLA 2010-10-06 10:56:30 EDT 
after some investigation, it seems that we have to use the RetargetableActions to redefine the default "move" action of the file menu

trying to understand all it works, on it
Comment 3 Bozier jerome CLA 2010-11-30 04:30:08 EST 
Created attachment 184099 [details]
patch

this patch fix the problem of move action in file menu not synchronized with file action from contextual menu
Comment 4 Bozier jerome CLA 2010-11-30 04:32:05 EST 
updating worked hours (long time for a small patch, sorry, had to learn how all of this works before putting my hands inside)

Paul, could you review it ? 

many thanks in advance
Comment 5 Bozier jerome CLA 2010-11-30 05:54:38 EST 
manual test added under CVS and wiki updated
Comment 6 Bozier jerome CLA 2010-11-30 08:13:32 EST 
patch delivered under CVS after review
Comment 7 Paul Slauenwhite CLA 2010-12-07 09:33:22 EST 
Verified in TPTP-4.7.2-201012061900.

Closing.