[#974] Add component edit window tab remembrance

swing/src/net/sf/openrocket/gui/configdialog/ComponentConfigDialog.java

@@ -40,6 +40,8 @@ public class ComponentConfigDialog extends JDialog implements ComponentChangeLis
 	private RocketComponent component = null;
 	private RocketComponentConfig configurator = null;
 
+	private static String previousSelectedTab = null;	// Name of the previous selected tab
+	
 	private final Window parent;
 	private static final Translator trans = Application.getTranslator();
 	
@@ -103,6 +105,9 @@ public class ComponentConfigDialog extends JDialog implements ComponentChangeLis
 		this.setContentPane(configurator);
 		configurator.updateFields();
 
+		// Set the selected tab
+		configurator.setSelectedTab(previousSelectedTab);
+		
 		//// configuration
 		setTitle(trans.get("ComponentCfgDlg.configuration1") + " " + component.getComponentName() + " " + trans.get("ComponentCfgDlg.configuration"));
 		
@@ -204,11 +209,17 @@ public class ComponentConfigDialog extends JDialog implements ComponentChangeLis
 	 * @param document		the document to configure.
 	 * @param component		the component to configure.
 	 * @param listeners		config listeners for the component
+	 * @param rememberPreviousTab if true, the previous tab will be remembered and used for the new dialog
 	 */
 	public static void showDialog(Window parent, OpenRocketDocument document,
-			RocketComponent component, List<RocketComponent> listeners) {
+			RocketComponent component, List<RocketComponent> listeners, boolean rememberPreviousTab) {
-		if (dialog != null)
+		if (dialog != null) {
+			previousSelectedTab = dialog.getSelectedTabName();
 			dialog.dispose();
+		}
+		if (!rememberPreviousTab) {
+			previousSelectedTab = null;
+		}
 
 		dialog = new ComponentConfigDialog(parent, document, component, listeners);
 		dialog.setVisible(true);
@@ -220,19 +231,51 @@ public class ComponentConfigDialog extends JDialog implements ComponentChangeLis
 	/**
 	 * A singleton configuration dialog.  Will create and show a new dialog if one has not
 	 * previously been used, or update the dialog and show it if a previous one exists.
+	 * By default, the previous tab is remembered.
+	 *
+	 * @param document		the document to configure.
+	 * @param component		the component to configure.
+	 * @param listeners		config listeners for the component
+	 */
+	public static void showDialog(Window parent, OpenRocketDocument document,
+								  RocketComponent component, List<RocketComponent> listeners) {
+		ComponentConfigDialog.showDialog(parent, document, component, listeners, true);
+	}
+
+	/**
+	 * A singleton configuration dialog.  Will create and show a new dialog if one has not
+	 * previously been used, or update the dialog and show it if a previous one exists.
+	 *
+	 * @param document		the document to configure.
+	 * @param component		the component to configure.
+	 * @param rememberPreviousTab if true, the previous tab will be remembered and used for the new dialog
+	 */
+	public static void showDialog(Window parent, OpenRocketDocument document,
+								  RocketComponent component, boolean rememberPreviousTab) {
+		ComponentConfigDialog.showDialog(parent, document, component, null, rememberPreviousTab);
+	}
+
+	/**
+	 * A singleton configuration dialog.  Will create and show a new dialog if one has not
+	 * previously been used, or update the dialog and show it if a previous one exists.
+	 * By default, the previous tab is remembered.
 	 *
 	 * @param document		the document to configure.
 	 * @param component		the component to configure.
 	 */
 	public static void showDialog(Window parent, OpenRocketDocument document,
 								  RocketComponent component) {
-		ComponentConfigDialog.showDialog(parent, document, component, null);
+		ComponentConfigDialog.showDialog(parent, document, component, null, true);
+	}
+
+	static void showDialog(RocketComponent component, List<RocketComponent> listeners, boolean rememberPreviousTab) {
+		showDialog(dialog.parent, dialog.document, component, listeners, rememberPreviousTab);
 	}
 	
 	
 	/* package */
 	static void showDialog(RocketComponent component, List<RocketComponent> listeners) {
-		showDialog(dialog.parent, dialog.document, component, listeners);
+		showDialog(dialog.parent, dialog.document, component, listeners, true);
 	}
 
 	/* package */
@@ -257,4 +300,12 @@ public class ComponentConfigDialog extends JDialog implements ComponentChangeLis
 		return (dialog != null) && (dialog.isVisible());
 	}
 
+	public String getSelectedTabName() {
+		if (configurator != null) {
+			return configurator.getSelectedTabName();
+		} else {
+			return null;
+		}
+	}
+	
 }

swing/src/net/sf/openrocket/gui/configdialog/RocketComponentConfig.java

@@ -269,6 +269,26 @@ public class RocketComponentConfig extends JPanel {
 		return subPanel;
 	}
 
+	public String getSelectedTabName() {
+		if (tabbedPane != null) {
+			return tabbedPane.getTitleAt(tabbedPane.getSelectedIndex());
+		} else {
+			return "";
+		}
+	}
+
+	public void setSelectedTab(String tabName) {
+		if (tabbedPane != null) {
+			for (int i = 0; i < tabbedPane.getTabCount(); i++) {
+				if (tabbedPane.getTitleAt(i).equals(tabName)) {
+					tabbedPane.setSelectedIndex(i);
+					return;
+				}
+			}
+			tabbedPane.setSelectedIndex(0);
+		}
+	}
+	
 	protected JPanel instanceablePanel( Instanceable inst ){
 		JPanel panel = new JPanel( new MigLayout("fill"));
 		{ // Instance Count

swing/src/net/sf/openrocket/gui/main/ComponentAddButtons.java

@@ -488,7 +488,7 @@ public class ComponentAddButtons extends JPanel implements Scrollable {
 				}
 			}
 			
-			ComponentConfigDialog.showDialog(parent, document, component);
+			ComponentConfigDialog.showDialog(parent, document, component, false);
 		}
 	}