You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/05/30 00:31:00 UTC
svn commit: r542683 -
/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/gui/util/MenuFactory.java
Author: sebb
Date: Tue May 29 15:30:59 2007
New Revision: 542683
URL: http://svn.apache.org/viewvc?view=rev&rev=542683
Log:
Bug 42539 - HCI: Inconsistent position of "Remove" in contextual menu
Modified:
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/gui/util/MenuFactory.java
Modified: jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/gui/util/MenuFactory.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/gui/util/MenuFactory.java?view=diff&rev=542683&r1=542682&r2=542683
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/gui/util/MenuFactory.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/gui/util/MenuFactory.java Tue May 29 15:30:59 2007
@@ -135,11 +135,6 @@
public static void addEditMenu(JPopupMenu menu, boolean removable) {
addSeparator(menu);
- if (removable) {
- menu.add(makeMenuItem(JMeterUtils.getResString("remove"), //$NON-NLS-1$
- "Remove", ActionNames.REMOVE, //$NON-NLS-1$
- KeyStroke.getKeyStroke(KeyEvent.VK_DELETE, 0)));
- }
menu.add(makeMenuItem(JMeterUtils.getResString("cut"), //$NON-NLS-1$
"Cut", ActionNames.CUT, //$NON-NLS-1$
KeyStroke.getKeyStroke(KeyEvent.VK_X, KeyEvent.CTRL_MASK)));
@@ -151,6 +146,11 @@
KeyStroke.getKeyStroke(KeyEvent.VK_V, KeyEvent.CTRL_MASK)));
menu.add(makeMenuItem(JMeterUtils.getResString("paste_insert"), //$NON-NLS-1$
"Paste Insert", ActionNames.PASTE)); //$NON-NLS-1$
+ if (removable) {
+ menu.add(makeMenuItem(JMeterUtils.getResString("remove"), //$NON-NLS-1$
+ "Remove", ActionNames.REMOVE, //$NON-NLS-1$
+ KeyStroke.getKeyStroke(KeyEvent.VK_DELETE, 0)));
+ }
}
public static void addFileMenu(JPopupMenu menu) {
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org