fix(gui): normalize tk_scaling in ProgressWindow and defer focus_force to prevent segfault

- Normalize tk_scaling with * 72.0/96.0 and add max(1.0, ...) floor in
  ProgressWindow to match BaseWindow._get_dpi_scaling_factor behaviour
- Use calculate_scaled_geometry() in VehicleProjectCreatorWindow instead
  of inline scaling to keep geometry logic centralized
- Defer focus_force() via after(1, ...) in PopupWindow to avoid a
  segfault in Python 3.9 on Linux/X11 in headless CI environments
- Call update_idletasks() before winfo_width/height in GUI test to
  prevent flaky 1px assertions before geometry is realized

Addresses review comments from PR #1394

Author: amilcarlucas

ardupilot_methodic_configurator/frontend_tkinter_progress_window.py

@@ -46,7 +46,8 @@ def __init__(  # pylint: disable=too-many-arguments, too-many-positional-argumen
         try:
             dpi = self.progress_window.winfo_fpixels("1i")
             tk_scaling = float(self.progress_window.tk.call("tk", "scaling"))
-            dpi_scaling_factor = max(dpi / 96.0, tk_scaling)
+            normalized_tk_scaling = tk_scaling * 72.0 / 96.0
+            dpi_scaling_factor = max(1.0, dpi / 96.0, normalized_tk_scaling)
         except (tk.TclError, AttributeError):
             dpi_scaling_factor = 1.0
         self.progress_window.geometry(f"{round(width * dpi_scaling_factor)}x{round(height * dpi_scaling_factor)}")

ardupilot_methodic_configurator/frontend_tkinter_project_creator.py

@@ -135,9 +135,7 @@ def template_selection_callback(_widget: "DirectorySelectionWidgets") -> str:
 
         # Set dynamic window size based on number of settings
         window_height = 250 + (len(settings_metadata) * 23)
-        self.root.geometry(
-            f"{round(800 * self.dpi_scaling_factor)}x{round(window_height * self.dpi_scaling_factor)}"
-        )  # Set the window size
+        self.root.geometry(self.calculate_scaled_geometry(800, window_height))  # Set the window size
         self.center_window_on_screen(self.root)
 
         for setting_name, metadata in settings_metadata.items():

ardupilot_methodic_configurator/frontend_tkinter_usage_popup_window.py

@@ -114,7 +114,9 @@ def finalize_setup_popupwindow(
             popup_window.root.deiconify()
             popup_window.root.lift()
             popup_window.root.update()  # Ensure the window is fully rendered before setting focus
-            popup_window.root.focus_force()
+            # Defer focus_force to after the event loop has processed the deiconify/lift,
+            # avoiding a segfault in Python 3.9 on Linux (X11) in headless environments.
+            popup_window.root.after(1, popup_window.root.focus_force)
 
             # On macOS, grab_set() causes UI freeze (issue #1264), so skip it
             # On Windows/Linux, make the popup modal and give it focus

tests/gui_frontend_tkinter_parameter_editor.py

@@ -158,6 +158,7 @@ def test_show_about_window(self, mocker) -> None:  # pylint: disable=too-many-lo
             # Check window properties
             assert about_window.title() == "About"
             # Check that geometry has positive dimensions (exact size depends on DPI scaling)
+            about_window.update_idletasks()
             win_width = about_window.winfo_width()
             win_height = about_window.winfo_height()
             assert win_width >= 650, f"Window width {win_width} should be at least 650px (unscaled)"