This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 24168 - Comment In/Out actions have reversed names
Summary: Comment In/Out actions have reversed names
Status: CLOSED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 3.x
Hardware: PC Windows ME/2000
: P2 blocker (vote)
Assignee: issues@editor
URL:
Keywords: UI
Depends on:
Blocks:
 
Reported: 2002-05-29 09:41 UTC by iformanek
Modified: 2007-11-05 13:42 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Part of the fix (1.72 KB, patch)
2002-07-15 16:02 UTC, Miloslav Metelka
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description iformanek 2002-05-29 09:41:30 UTC
Comment Out a piece of code means to put a comment around 
it so that it does not work. The Comment Out action does 
the reverse in the editor toolbar.
Comment 1 iformanek 2002-06-04 14:40:09 UTC
Confirmed by american (Chris) and australian (Jack) native 
speakers, Comment Out means to put comment around 
something to make it ineffective. Comment In is not an 
english phrase.
Comment 2 Gabriel Tichy 2002-06-04 17:08:26 UTC
OK, lets change it simply to Comment/Uncomment now.
Comment 3 Gabriel Tichy 2002-06-04 17:12:34 UTC
Fixed. Changed to "Comment" and "Uncomment".
Comment 4 iformanek 2002-07-15 13:04:41 UTC
Sorry to be a pest, but I cannot help myself - the icons 
on the comment and uncomment are still reversed, why does 
the uncomment action display the "//" and the comment one 
does not? Should it perhaps be opposite?
Comment 5 Miloslav Metelka 2002-07-15 16:01:24 UTC
I see - this was the effect of replacing the old icons with the new
ones from Bruce.

I have renamed the comment_out.gif and comment_in.gif
to comment.gif and uncomment.gif which will hopefully avoid further
confusion.

The fix:
1) rename comment_out.gif to comment.gif and comment_in.gif to
uncomment.gif in src/org/netbeans/modules/editor/resources/
2) cvs remove comment_out.gif comment_in.gif
3) cvs add comment.gif uncomment.gif
4) apply the attached fix

Comment 6 Miloslav Metelka 2002-07-15 16:02:40 UTC
Created attachment 6670 [details]
Part of the fix
Comment 7 Miloslav Metelka 2002-07-15 16:04:49 UTC
Integrated into main trunk

Checking in libsrc/org/netbeans/editor/ext/ExtKit.java;
/cvs/editor/libsrc/org/netbeans/editor/ext/ExtKit.java,v  <--  ExtKit.java
new revision: 1.30; previous revision: 1.29
done
Processing log script arguments...
More commits to come...
RCS file:
/cvs/editor/src/org/netbeans/modules/editor/resources/comment.gif,v
done
Checking in src/org/netbeans/modules/editor/resources/comment.gif;
/cvs/editor/src/org/netbeans/modules/editor/resources/comment.gif,v 
<--  comment.gif
initial revision: 1.1
done
Removing src/org/netbeans/modules/editor/resources/comment_in.gif;
/cvs/editor/src/org/netbeans/modules/editor/resources/comment_in.gif,v
 <--  comment_in.gif
new revision: delete; previous revision: 1.3
done
Removing src/org/netbeans/modules/editor/resources/comment_out.gif;
/cvs/editor/src/org/netbeans/modules/editor/resources/comment_out.gif,v
 <--  comment_out.gif
new revision: delete; previous revision: 1.3
done
RCS file:
/cvs/editor/src/org/netbeans/modules/editor/resources/uncomment.gif,v
done
Checking in src/org/netbeans/modules/editor/resources/uncomment.gif;
/cvs/editor/src/org/netbeans/modules/editor/resources/uncomment.gif,v
 <--  uncomment.gif
initial revision: 1.1
done
Comment 8 Jan Lahoda 2002-07-16 14:27:31 UTC
It seems OK to me in 200207160100.
Comment 9 Miloslav Metelka 2002-07-17 17:27:57 UTC
Adding a diff hyperlink to the changed ExtKit:

http://editor.netbeans.org/source/browse/editor/libsrc/org/netbeans/editor/ext/ExtKit.java.diff?r1=1.29&r2=1.30

Mato, could you please review the fix? Thanks.
Comment 10 Miloslav Metelka 2002-07-18 08:51:05 UTC
Fixed in trunk.
Comment 11 Martin Roskanin 2002-07-18 09:47:41 UTC
Reviewed. The fix is OK. Mato
Comment 12 Miloslav Metelka 2002-07-23 14:40:38 UTC
Integrated into release34.
Comment 13 Jan Lahoda 2002-07-24 09:38:28 UTC
Verified in 200207232340.
Comment 14 Quality Engineering 2003-06-30 18:11:49 UTC
Resolved for 3.3.x or earlier, no new info since then -> closing.