Bug 219091 - source folder name in extracted files
Summary: source folder name in extracted files
Status: CLOSED FIXED
Alias: None
Product: Babel
Classification: Technology
Component: Server (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Babel server inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-15 09:02 EST by Kit Lo CLA
Modified: 2008-03-07 15:39 EST (History)
0 users

See Also:

Attachments
source folder name in extracted files (5.55 KB, image/x-png)
2008-02-15 09:02 EST, Kit Lo CLA 		no flags Details
patch (833 bytes, patch)
2008-02-28 15:56 EST, Denis Roy 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 Kit Lo CLA 2008-02-15 09:02:40 EST 
Created attachment 89840 [details]
source folder name in extracted files

Folder name is currently included in the extracted file name. Some plugins, like ant.ui, are using spaces in their source folder names. It's causing some wrapping in the Files list on the server. The files list is very hard to read! See screen capture.

I wonder if the "folder name" is really needed. Can't we have dir structure like: org.eclipse.ant.ui/org/eclipse/ant/internal/ui/AntUIModelMessages.properties? We should be able to build the NL packs without the "folder name".

Another suggestion for the translation tool is to add an ellipsis if the name is longer than X characters, like: org.eclipse.ant.ui/(...)/ui/AntUIModelMessages.properties, or to add a horizontal scroll bar in the pane (not sure if that's possible).
Comment 1 Denis Roy CLA 2008-02-28 15:56:02 EST 
Created attachment 91050 [details]
patch

Here's a patch to fix this.

Note this patch overlaps work done on Bug 217261.
Comment 2 Denis Roy CLA 2008-02-29 13:15:39 EST 
I've committed this to the staging area.  Closing as fixed; please reopen if this isn't what you were expecting.
Comment 3 Kit Lo CLA 2008-03-07 15:38:47 EST 
confirmed that ellipsis is added to long file names
Comment 4 Kit Lo CLA 2008-03-07 15:39:00 EST 
closing