Bug 312320 - TVT36:TCT199: Mnemonics on TestConnection context menu are inconsistent with Eclipse Platform
Summary: TVT36:TCT199: Mnemonics on TestConnection context menu are inconsistent with ...
Status: CLOSED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows 7
: P5 trivial (vote)
Target Milestone: 3.2 RC1   Edit
Assignee: Martin Oberhuber CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: PII
Depends on:
Blocks:
 
Reported: 2010-05-10 14:50 EDT by CDE Administration CLA
Modified: 2010-05-28 11:21 EDT (History)
3 users (show)

See Also:
xuanchen: review+


Attachments
14.000.230-UnableToGetUniqueMnemonics.gif (327.10 KB, image/gif)
2010-05-10 14:50 EDT, CDE Administration CLA
no flags Details
Patch v1 (3.69 KB, patch)
2010-05-11 08:45 EDT, Martin Oberhuber CLA
no flags Details | Diff
TCT199_mnemo.gif (339.09 KB, image/gif)
2010-05-27 12:35 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 2010-05-10 14:50:36 EDT
<response_by> martine marin at 2010.05.07.08.32.34 </response_by>
Hi,

In the context menu when you right-click TestConnection it is impossible to get unique mnemonic, it also seems that the English ones are duplicate for instance for Rename and Monitor (on M). Most of these strings appear also elsewhere and if you change in this view they won't be correct in another view, etc.

Steps to reproduce:

In the Remote Systems view, right-click on TestConnection

<response_by> Kathy Li at 2010.05.10.13.31.35 </response_by>
Lower priority to 3 because it is mnemonics related issues.

<response_by> Kathy Li at 2010.05.10.13.37.12 </response_by>
This article was reassigned from Category:''TVT/Testing,Inbox''.
Comment 1 CDE Administration CLA 2010-05-10 14:50:45 EDT
Created attachment 167782 [details]
14.000.230-UnableToGetUniqueMnemonics.gif
Comment 2 CDE Administration CLA 2010-05-10 14:50:57 EDT
<cde:tctdetail>
Testcase: 14.000.230 - RSE - MENUS
Project: WSW36
Component: Xfer - Target Management/RSE
Priority: 3
Subject: FRA - Unable to get unique mnemonics on TestConnection context menu
Article ID: 199
Originator: mmarin@fr.ibm.com
</cde:tctdetail>
Comment 3 Martin Oberhuber CLA 2010-05-11 07:54:11 EDT
Changed summary, previous value was:
TVT36:TCT199: FRA - Unable to get unique mnemonics on TestConnection context menu

First of all, duplicate menmonics in non-English RSE would not be an Open Source issue but a problem for the company proviging translations. In this case, what matters is that the same problem exists in the English version of RSE, so I'm accepting the case.

Second, duplicate menmonics are usually not an issue -- just typing the mnemonic char twice works just fine. What's most important is that mnemonics are deterministic (always the same), which doesn't seem the case here (In one instance I have seen "Monitor" both with an "M" mnemonic and with a "T" mnemonic). 

In general, because Eclipse is Open to extensions, there is no way guaranteeing consistent, deterministic, non-conflicting mnemonics everywhere. The one thing that can be done is assigning a fixed mnemonic for often-used actions which appear in multiple different, unrelated menus. Assigning too many fixed mnemonics again is a recipe for conflicts in other places.

In the concrete case, the only useful change that I could see would be
1.) Change Mnemonic for "Move up" from (V) to (U) -- direction ("up") matters
2.) Change Mnemonic for "Move" from (O) to (V) -- same as File>Move
3.) Change Mnemonic for "Monitor" from (M) to (O) or (T), or keep unassigned

As you see, assignment of mnemonics is tricky and there is good chance that one change in a place introduces conflicts in another place. I'm not even sure whether a change to PII for such a minor issue is acceptable that late in the game (we are at RC1).
Comment 4 Martin Oberhuber CLA 2010-05-11 08:45:19 EDT
Created attachment 167906 [details]
Patch v1

Attached patch fixes
  Move Up --> Move &Up
  Move    --> Mo&ve
  Go Into --> Go &Into
  Monitor --> Moni&tor

Since these are all default hard-coded mnemonics also found in Eclipse Platform:

messages.properties - org.eclipse.ui.ide/src/org/eclipse/ui/internal/views/framelist

and thus make the RSE view consistent with other Eclipse views.
Comment 5 Martin Oberhuber CLA 2010-05-11 08:48:50 EDT
I changed the summary again, since I believe the most important fix of my patch is making mnemonics consistent with the Eclipse Platform (which should also make translation easier, at least for Babel).

Xuan please let me know whether this fix (which affects PII) is acceptable for 3.2RC1.
Comment 6 Xuan Chen CLA 2010-05-11 10:10:24 EDT
Thanks for looking into this, Martin.
I checked the NL schedule, and we still have time to make PII changes. Thanks.
Comment 7 Martin Oberhuber CLA 2010-05-11 10:27:35 EDT
Committed for 3.2RC1.

I'm sure that there are many similar cases since none of the labels in SystemResources.properties use any hard-coded mnemonics today. But I don't have time to review all of them for consistency with Eclipse Platform labels / mnemonics.
Comment 8 CDE Administration CLA 2010-05-25 12:21:03 EDT
<response_by> Kathy Li at 2010.05.25.11.09.51 </response_by>
Hi Xuan, any update on this one? Any plan to fix the issues?
Thanks
Kathy
Comment 9 Martin Oberhuber CLA 2010-05-25 14:07:02 EDT
Look at the status. It is fixed as per TM 3.2rc1.
Comment 10 CDE Administration CLA 2010-05-25 15:06:17 EDT
<response_by> Kathy Li at 2010.05.25.13.56.15 </response_by>
Thank you for the confirmation, we'll ask countries to verify.
Comment 11 CDE Administration CLA 2010-05-27 12:35:55 EDT
<response_by> martine marin at 2010.05.27.11.34.17 </response_by>
Hi,

This is impossible to get them all unique.
For Properties since there is a workaround Alt+Enter I guess it's OK to have it duplicate.

When you have more than 15 items it is really impossible to fix all mnemonics, for Monitor, all the letters are already taken for instance.

Verified with build 20100526-1600

Thanks,

Martine
Comment 12 CDE Administration CLA 2010-05-27 12:35:57 EDT
Created attachment 170220 [details]
TCT199_mnemo.gif
Comment 13 Martin Oberhuber CLA 2010-05-27 13:16:17 EDT
Why do you reopen when you write it's verified?
Comment 14 CDE Administration CLA 2010-05-28 11:21:22 EDT
<response_by> Kathy Li at 2010.05.28.10.19.00 </response_by>
I'm closing it as it's verified by FR.
Comment 15 CDE Administration CLA 2010-05-28 11:21:28 EDT
[Added by CDE: Completed by the CDE Bridge]