Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 88249

Summary: TVT3.1 #48 - RUS "Space" missing in menu
Product: [Tools] CDT Reporter: David W Hare <dwhare>
Component: cdt-coreAssignee: Project Inbox <cdt-core-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: alexander.glotov, camle, shihmy, twolff
Version: 3.0   
Target Milestone: ---   
Hardware: PC   
OS: Linux-GTK   
Whiteboard:
Attachments:
Description Flags
Russian Screenshot
none
English screenshot none

Description David W Hare CLA 2005-03-16 16:26:27 EST
Open a .cpp file in the C/C++ Projects view.  
Click the Edit menu.  

You'll see in an English locale that for "Content Assist" the key combination is
"CTRL+Space".  

In the Russian locale, it displays as just "CTRL+".  The word Space has been
translated and is not displaying.  

Is variable substitution being used here?

See screenshots...
Comment 1 David W Hare CLA 2005-03-16 16:27:17 EST
Created attachment 18875 [details]
Russian Screenshot
Comment 2 David W Hare CLA 2005-03-16 16:28:10 EST
Created attachment 18876 [details]
English screenshot
Comment 3 Kim Horne CLA 2005-03-19 03:24:24 EST
*** Bug 87131 has been marked as a duplicate of this bug. ***
Comment 4 Alexander Glotov CLA 2005-03-22 10:51:10 EST
The string used for the menu item itself is defined in 
\eclipse\plugins\org.eclipse.cdt.ui\cdtui_jar\org\eclipse\cdt\internal\ui\editor\CEditorMessages.properties
as ContentAssistProposal.label=Co&ntent Assist@Ctrl+Space
This can easily be proven by modifying this string in the live jar file
to ContentAssistProposal.label=TEST Co&ntent Assist@Ctrl+Space
and looking at the menu item in C++ Editor. 

But if we change the string in the jared file to something like 
ContentAssistProposal.label=Co&ntent Assist@WHEREISIT Ctrl+Space
we won't see this "WHEREISIT" in the menu item text.
This means that menu item in the C++ editor is assembled from 
"Co&ntent Assist" taken from ContentAssistProposal.label file in the 
\eclipse\plugins\org.eclipse.cdt.ui\cdtui_jar\org\eclipse\cdt\internal\ui\editor\CEditorMessages.properties
and "Ctrl+Space" from some other file, namely
\eclipse\plugins\org.eclipse.ui.workbench\workbench_jar\org\eclipse\ui\internal\keys\AbstractKeyFormatter.properties
(keys CTRL=Ctrl and SPACE=Space)

We do have all "Space" and "Ctrl+Space" strings translated now, so the only
question so far is if it is possible to display those strings correctly in this
particular place (they are displayed correctly in all other places).

Alexander Glotov
Comment 5 David Daoust CLA 2005-03-25 19:19:47 EST
You should be able to drop the stuff that is after the @ in this example

i.e.
ContentAssistProposal.label=Co&ntent Assist

The keybinding was redundant (likely a holdover from an earlier version of
eclipse).  It has already been changed to this in the HEAD stream of CDT.
This is related to: https://bugs.eclipse.org/bugs/show_bug.cgi?id=69157 
Comment 6 David W Hare CLA 2005-03-29 13:48:24 EST
The Russian translation has been fixed.
Comment 7 David W Hare CLA 2005-03-29 13:48:40 EST
closing