diff --git a/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurable.java b/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurable.java index 51ab7fdf97dc..30f79a225ed1 100644 --- a/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurable.java +++ b/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurable.java @@ -26,10 +26,7 @@ import com.intellij.openapi.project.Project; import com.intellij.openapi.ui.LabeledComponent; import com.intellij.openapi.ui.TextFieldWithBrowseButton; import com.intellij.openapi.util.text.StringUtil; -import com.intellij.ui.EditorTextFieldWithBrowseButton; -import com.intellij.ui.PanelWithAnchor; -import com.intellij.ui.RawCommandLineEditor; -import com.intellij.ui.ScrollPaneFactory; +import com.intellij.ui.*; import com.intellij.ui.components.JBLabel; import com.intellij.ui.table.TableView; import com.intellij.util.ui.ColumnInfo; @@ -162,19 +159,18 @@ public class AppletConfigurable extends SettingsEditor impl } private void removeParameter() { - final int selectedRow = myTable.getSelectedRow(); - if (selectedRow < 0 || selectedRow >= myTable.getRowCount()) return; - final ArrayList newItems = - new ArrayList(myParameters.getItems()); - newItems.remove(selectedRow); - myParameters.setItems(newItems); + TableUtil.removeSelectedItems(myTable); } private void addParameter() { - final ArrayList newItems = - new ArrayList(myParameters.getItems()); - newItems.add(new AppletConfiguration.AppletParameter("newParameter", "")); + final ArrayList newItems = new ArrayList(myParameters.getItems()); + final AppletConfiguration.AppletParameter parameter = new AppletConfiguration.AppletParameter("newParameter", ""); + newItems.add(parameter); myParameters.setItems(newItems); + + int index = newItems.size() - 1; + myTable.getSelectionModel().setSelectionInterval(index, index); + myTable.scrollRectToVisible(myTable.getCellRect(index, 0, true)); } private JComboBox getModuleComponent() { @@ -185,14 +181,9 @@ public class AppletConfigurable extends SettingsEditor impl return myPolicyFile.getComponent(); } - private void getConfigurationTo(final AppletConfiguration configuration) { - - } - - private List cloneParameters(final List items) { + private static List cloneParameters(final List items) { final List params = new ArrayList(); - for (Iterator iterator = items.iterator(); iterator.hasNext();) { - AppletConfiguration.AppletParameter appletParameter = iterator.next(); + for (AppletConfiguration.AppletParameter appletParameter : items) { params.add(new AppletConfiguration.AppletParameter(appletParameter.getName(), appletParameter.getValue())); } return params; @@ -311,15 +302,6 @@ public class AppletConfigurable extends SettingsEditor impl super(name); } - public Comparator getComparator() { - return new Comparator() { - public int compare(final AppletConfiguration.AppletParameter parameter1, - final AppletConfiguration.AppletParameter parameter2) { - return valueOf(parameter1).compareTo(valueOf(parameter2)); - } - }; - } - public TableCellEditor getEditor(final AppletConfiguration.AppletParameter item) { final JTextField textField = new JTextField(); textField.setBorder(BorderFactory.createLineBorder(Color.BLACK));