321648 – No "Source" menu / context menu for IMP editors

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

Bug 321648 - No "Source" menu / context menu for IMP editors

Summary: No "Source" menu / context menu for IMP editors

Status:	RESOLVED FIXED

Alias:	None

Product:	z_Archived
Classification:	Eclipse Foundation
Component:	IMP (show other bugs)
Version:	unspecified
Hardware:	All All

Importance:	P3 enhancement (vote)
Target Milestone:	---
Assignee:	Robert M. Fuhrer
QA Contact:

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-08-03 15:11 EDT by Adam Taylor
Modified:	2014-01-09 15:04 EST (History)
CC List:	0 users

See Also:

Attachments
Here's a possible implementation based upon JDT (39.23 KB, patch) 2010-08-04 12:20 EDT, Adam Taylor	no flags	Details \| Diff
SWTBot test plugin for the proposed patch (9.38 KB, application/octet-stream) 2010-08-04 12:22 EDT, Adam Taylor	no flags	Details
missing test plugin dependency (18.56 KB, application/octet-stream) 2010-08-04 14:39 EDT, Adam Taylor	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 Adam Taylor

2010-08-03 15:11:35 EDT

Build Identifier: 

When in a Java Editor there is a "Source" main menu and a "Source" context menu which contains actions such as toggle comment, indent selection, etc.  This is not present for an IMP editor

Reproducible: Always

Steps to Reproduce:
Open IMP editor

Comment 1 Adam Taylor

2010-08-04 12:20:59 EDT

Created attachment 175848 [details]
Here's a possible implementation based upon JDT

Comment 2 Adam Taylor

2010-08-04 12:22:48 EDT

Created attachment 175850 [details]
SWTBot test plugin for the proposed patch

Comment 3 Adam Taylor

2010-08-04 14:39:28 EDT

Created attachment 175871 [details]
missing test plugin dependency

Comment 4 Robert M. Fuhrer

2010-08-12 16:04:45 EDT

This patch looks like it was all copied/adapted from the JDT. Was any part of this taken from anywhere else?

Comment 5 Adam Taylor

2010-08-12 16:24:43 EDT

That's correct, it all came from the JDT.

Comment 6 Robert M. Fuhrer

2010-09-09 18:35:47 EDT

Reviewed patches; they all looked reasonable to me. Applied to HEAD of SVN trunk. Will go out as part of the next IMP release.

Thanks!