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

Bug 362102

Summary: [nls tooling][quick assist] Improve preview for 'Remove property'
Product: [Eclipse Project] JDT Reporter: Markus Keller <markus.kell.r>
Component: TextAssignee: Deepak Azad <deepakazad>
Status: CLOSED DUPLICATE QA Contact:
Severity: normal    
Priority: P3 CC: daniel_megert, markus.kell.r, raksha.vasisht
Version: 3.8   
Target Milestone: 3.8 M4   
Hardware: All   
OS: All   
Whiteboard:

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 ***