Bug 235714 - [KeyBindings] TVT34:TCT738: TVT FR - CDT - Space missing before colon sign
Summary: [KeyBindings] TVT34:TCT738: TVT FR - CDT - Space missing before colon sign
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4   Edit
Hardware: PC Linux-GTK
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL: 738
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-04 17:16 EDT by CDE Administration CLA
Modified: 2019-09-06 16:11 EDT (History)
3 users (show)

See Also:


Attachments
05_001650.gif (278.67 KB, image/gif)
2008-06-04 17:16 EDT, CDE Administration CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description CDE Administration CLA 2008-06-04 17:16:42 EDT
<response_by> martine marin at 2008.06.04.14.20.53 </response_by>
OS: Linux/Windows
Build: 0602
Component: CDT
testcase: 05.001650
Steps to reproduce
Select General -> Keys.
Type C++ to filter out non-CDT key bindings

There is a space missing before the colon sign in expression between parenthesis (punctuation seemed to be added by the system)

Thanks,

Martine

<response_by> Karl Mittmann at 2008.06.04.15.59.55 </response_by>
This article was reassigned from Category:''TVT/Testing,Inbox''.

<response_by> Karl Mittmann at 2008.06.04.16.00.20 </response_by>
This comment was made to a similar previous case,

"I will add special processing to our colon handling for this release for French locales. This will be removed in next release when we will remove all specialized colon processing. See bug 231675."
Comment 1 CDE Administration CLA 2008-06-04 17:16:52 EDT
Created attachment 103650 [details]
05_001650.gif
Comment 2 CDE Administration CLA 2008-06-04 17:16:59 EDT
<cde:tctdetail>
Testcase: 05.001650
Project: WSW34
Component: Xfer - CDT/UI
Priority: 2
Subject: TVT FR - CDT - Space missing before colon sign
Article ID: 738
Originator: mmarin@fr.ibm.com
</cde:tctdetail>
Comment 3 Vivian Kong CLA 2008-06-05 11:15:02 EDT
The strings in question are used in an extension point.  So the concatenation of the strings (with the colon) are done by Platform.
Comment 4 Paul Webster CLA 2008-06-06 08:48:22 EDT
That's not part of a sentence (and hence no grammar).  It's simply specifying optional command parameters->possible values (map of tuples).

Unless there's some specific examples of this that need to work differently, this is WONTFIX.

PW
Comment 5 Kit Lo CLA 2008-06-06 09:34:10 EDT
Paul, do you know if the parenthesis and colon are inserted programmatically or not?

Is it possible to externalize a string like "{0} ({1}: {2})" and use MessageFormat to subsitute {0}, {1} and {2} with commandName, commandParm, and value?

That way, if a language chooses to, they can add the space in front of the colon.
Comment 6 Paul Webster CLA 2008-06-06 09:51:41 EDT
I could allow ParameterizedCommand getName() to use a ResourceBundle for the patterns in 3.5.

(parm, parm, parm) is one pattern, and for each individual parm it is parmName: parmValue

If it is a standard in French that these kind of tuple maps are parmName : parmValue.

PW
Comment 7 Kit Lo CLA 2008-06-06 10:03:16 EDT
Thanks Paul, that sounds good. Can that be done in 3.4.1?
Comment 8 Paul Webster CLA 2008-06-06 10:08:35 EDT
(In reply to comment #7)
> Thanks Paul, that sounds good. Can that be done in 3.4.1?
> 

I'm waiting to hear that parmName : parmValue is a requirement in French first (i.e. no one would ever write parmName: parmValue).

But no, it would not be done until 3.5

PW
Comment 9 CDE Administration CLA 2008-06-09 10:18:23 EDT
<response_by> martine marin at 2008.06.09.09.10.39 </response_by>
Can you give a status Deferred for this defect?
Thanks a lot in advance,
Martine
Comment 10 CDE Administration CLA 2008-06-09 15:17:38 EDT
<response_by> martine marin at 2008.06.09.14.07.47 </response_by>
Hi,

Yes this is standard in French, there is always a space in front of the colon sign in French (also for question marks and other punctuation items).
Please fix and/or defer this defect.
Thanks,
Martine
Comment 11 CDE Administration CLA 2008-06-12 11:27:05 EDT
<response_by> John Ryding at 2008.06.12.10.13.05 </response_by>
This has been deferred. It will be considered for the 3.4.1 service pack.
Comment 12 Paul Webster CLA 2009-03-02 11:41:41 EST
Updated as per http://wiki.eclipse.org/Platform_UI/Bug_Triage
PW
Comment 13 Eclipse Webmaster CLA 2019-09-06 16:11:23 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.