Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 362102 - [nls tooling][quick assist] Improve preview for 'Remove property'
Summary: [nls tooling][quick assist] Improve preview for 'Remove property'
Status: CLOSED DUPLICATE of bug 361916
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.8   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.8 M4   Edit
Assignee: Deepak Azad CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-26 13:06 EDT by Markus Keller CLA
Modified: 2011-10-27 02:54 EDT (History)
3 users (show)

See Also:

Attachments
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 Keller CLA 2011-10-26 13:06:25 EDT 
Improve the preview for 'Remove property' and 'Remove properties' proposals.

The current preview doesn't show anything interesting. Check how it looks if you render the removed properties with strikeout style. Or put an explicit "<removed property>" there.

Furthermore, this is the first quick assist that makes changes in multiple files. The preview has to show where the changes will be performed (e.g. with "Filename.ext:" in front of the section).
Comment 1 Raksha Vasisht CLA 2011-10-26 15:35:45 EDT 
(In reply to comment #0)
> Improve the preview for 'Remove property' and 'Remove properties' proposals.
> 
> The current preview doesn't show anything interesting. Check how it looks if
> you render the removed properties with strikeout style. Or put an explicit
> "<removed property>" there.
> 
> Furthermore, this is the first quick assist that makes changes in multiple
> files. The preview has to show where the changes will be performed (e.g. with
> "Filename.ext:" in front of the section).

Dup of bug 361916?
Comment 2 Dani Megert CLA 2011-10-27 02:54:22 EDT 

*** This bug has been marked as a duplicate of bug 361916 ***