Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315557 - [wording] Improve naming of 'Convert Line Delimiters To > ...' actions
Summary: [wording] Improve naming of 'Convert Line Delimiters To > ...' actions
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.6   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.7 M1   Edit
Assignee: Markus Keller CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-03 07:07 EDT by Markus Keller CLA
Modified: 2010-08-03 09:16 EDT (History)
2 users (show)

See Also:

Attachments
Fix (7.77 KB, patch)
2010-06-04 07:17 EDT, Markus Keller CLA 		no flags Details | Diff
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 Markus Keller CLA 2010-06-03 07:07:03 EDT 
Improve naming of 'Convert Line Delimiters To > ...' actions, so that they can  also serve as reference for the platform conventions.

The names should tell exactly what delimiters will be used, e.g.:

Windows (CRLF, \r\n, 0D0A, ¤¶)
Unix (LF, \n, 0A, ¶)
Mac OS 9 (CR, \r, 0D, ¤)

The last symbol is the one shown in text editors when Show Whitespace Characters is enabled.
Comment 1 Dani Megert CLA 2010-06-04 02:44:07 EDT 
> Windows (CRLF, \r\n, 0D0A, ¤¶)
> Unix (LF, \n, 0A, ¶)
> Mac OS 9 (CR, \r, 0D, ¤)

The menu entries might get a bit long. Will see how it looks.
Comment 2 Markus Keller CLA 2010-06-04 05:57:53 EDT 
I've already started to fix this, will attach a patch.

The menu entries look good to me, but the command names get a bit long, but you get the added benefit that e.g. CRLF in the Quick Access dialog or the Keys preference page finds the command.
Comment 3 Dani Megert CLA 2010-06-04 05:59:26 EDT 
>I've already started to fix this, will attach a patch.
Good for me :-)
Comment 4 Markus Keller CLA 2010-06-04 07:17:17 EDT 
Created attachment 171085 [details]
Fix
Comment 5 Markus Keller CLA 2010-06-23 10:00:07 EDT 
Fixed in HEAD.
Comment 6 Deepak Azad CLA 2010-08-03 09:16:43 EDT 
Verified with I20100802-1800 on Ubuntu.