Bug 163253 - [editor] Remove Block comment: End comment tag not removed by "Remove Block Comment"
Summary: [editor] Remove Block comment: End comment tag not removed by "Remove Block C...
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.sse (show other bugs)
Version: 1.0.3   Edit
Hardware: PC Windows XP
: P3 trivial (vote)
Target Milestone: ---   Edit
Assignee: wst.sse CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on: 86520
Blocks:
  Show dependency tree
 
Reported: 2006-11-02 15:10 EST by Brian Jakubik CLA
Modified: 2010-12-15 16:01 EST (History)
2 users (show)

See Also:

Attachments
File to use for the test (438 bytes, text/xml)
2006-11-02 15:11 EST, Brian Jakubik 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 Brian Jakubik CLA 2006-11-02 15:10:25 EST 
This was reported by one of our users, and I am forwarding it on.

The "Remove Block Comment" command does not remove the end comment tag if the following steps are followed (Please use attached test script):

1. Highlight lines 8-13 (includes onentry & onexit tags)
2. Select add block comment from the source menu (block comment is added)
3. Right-click on document and select Format -> Document (The starting and ending comment tags are moved to a separate line)
4. Highlight lines 6-13 (begins with the line containing "<!--" and ends with line containing "-->"
5. Select "Remove Block Comment" from the Source menu. 

Result: The end comment tag is not removed.

It may be related to https://bugs.eclipse.org/bugs/show_bug.cgi?id=109051
but Im not quite sure.

Seems a little quirky, but not that big of a deal
Comment 1 Brian Jakubik CLA 2006-11-02 15:11:57 EST 
Created attachment 53159 [details]
File to use for the test

Attached is the file that was used by the user when they found the problem.
Comment 2 Amy Wu CLA 2008-10-27 04:34:35 EDT 
reassigning to inbox
Comment 3 Ian Tewksbury CLA 2009-07-07 14:24:10 EDT 
This will be fixed by Bug 86520
Comment 4 Ian Tewksbury CLA 2010-01-18 17:03:03 EST 
Will be fixed by Bug 86520.
Comment 5 Nick Sandonato CLA 2010-12-15 16:01:49 EST 
Resolved by Bug 86520.