Merge branch 'unstable' into issue-2357

ReleaseNotes.md

@@ -75,7 +75,7 @@ This is the first beta release of version 23.09.
 * Show calculated values in override tab (fixes #1629)
 * Decrease minimum FoV to 10 degrees in Photo Studio
 * Increase resolution of launch temperature and pressure to 2 decimal places (fixes #2003)
-* Display Cd oerride with 3 decimal places
+* Display Cd override with 3 decimal places
 * Add wiki button to help menu (fixes #2046)
 * Eliminate option to save "some" sim data (fixes #2024)
 * Add OK/Cancel buttons when editing simulations (fixes #2158)

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

@@ -303,14 +303,21 @@ public class RocketComponentConfig extends JPanel {
 				}
 
 				// Yes/No dialog: Are you sure you want to discard your changes?
+				SwingUtilities.invokeLater(() -> {
 					JPanel msg = createCancelOperationContent();
 					int resultYesNo = JOptionPane.showConfirmDialog(RocketComponentConfig.this, msg,
 							trans.get("RocketCompCfg.CancelOperation.title"), JOptionPane.YES_NO_OPTION, JOptionPane.WARNING_MESSAGE);
+
 					if (resultYesNo == JOptionPane.YES_OPTION) {
-					ComponentConfigDialog.clearConfigListeners = false;		// Undo action => config listeners of new component will be cleared
+						ComponentConfigDialog.clearConfigListeners = false;
+
+						// Need to execute after delay, otherwise the dialog will not be disposed
+						GUIUtil.executeAfterDelay(100, () -> {
 							disposeDialog();
 							document.undo();
+						});
 					}
+				});
 			}
 		});
 		buttonPanel.add(cancelButton, "split 2, right, gapleft 30lp");

swing/src/net/sf/openrocket/gui/dialogs/preferences/GeneralPreferencesPanel.java

@@ -364,7 +364,11 @@ public class GeneralPreferencesPanel extends PreferencesPanel {
 									trans.get("generalprefs.ImportWarning.msg"),
 									trans.get("generalprefs.ImportWarning.title"),
 									JOptionPane.WARNING_MESSAGE);
+
+							// Need to execute after delay, otherwise the dialog will not be disposed
+							GUIUtil.executeAfterDelay(100, () -> {
 								PreferencesDialog.showPreferences(parent.getParentFrame());		// Refresh the preferences dialog
+							});
 						}
 					});
 				}

swing/src/net/sf/openrocket/gui/dialogs/preset/ComponentPresetTable.java

@@ -86,7 +86,10 @@ public class ComponentPresetTable extends JTable {
 				ComponentPreset preset = ComponentPresetTable.this.presets.get(rowIndex);
 				Application.getComponentPresetDao().setFavorite(preset, presetType, (Boolean) aValue);
 				ComponentPresetTable.this.updateFavorites();
-				ComponentPresetTable.this.setRowSelectionInterval(rowIndex, rowIndex);
+				int viewIndex = ComponentPresetTable.this.convertRowIndexToView(rowIndex);
+				if (viewIndex != -1) {
+					ComponentPresetTable.this.setRowSelectionInterval(viewIndex, viewIndex);
+				}
 			}
 
 			@Override

swing/src/net/sf/openrocket/gui/util/GUIUtil.java

@@ -51,6 +51,7 @@ import javax.swing.RootPaneContainer;
 import javax.swing.SpinnerModel;
 import javax.swing.SpinnerNumberModel;
 import javax.swing.SwingUtilities;
+import javax.swing.Timer;
 import javax.swing.UIManager;
 import javax.swing.border.TitledBorder;
 import javax.swing.event.ChangeListener;
@@ -732,4 +733,21 @@ public class GUIUtil {
 		
 	}
 
+	/**
+	 * Executes the given code after a specified delay.
+	 *
+	 * @param delayMillis the delay in milliseconds.
+	 * @param runnable the code to be executed after the delay.
+	 */
+	public static void executeAfterDelay(int delayMillis, Runnable runnable) {
+		Timer timer = new Timer(delayMillis, new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent arg0) {
+				runnable.run();
+			}
+		});
+		timer.setRepeats(false);
+		timer.start();
+	}
+	
 }