Test custom centreing again

resources/wx_gui/gui_about.py

@@ -18,7 +18,7 @@ class AboutFrame(wx.Frame):
         logging.info("Generating About frame")
         super(AboutFrame, self).__init__(None, title="About", size=(350, 350), style=wx.DEFAULT_FRAME_STYLE & ~(wx.RESIZE_BORDER | wx.MAXIMIZE_BOX))
         self.constants: constants.Constants = global_constants
-        self.Centre()
+        gui_support.Centre(self, self.constants)
         self.hyperlink_colour = (25, 179, 231)
 
         self._generate_elements(self)

resources/wx_gui/gui_build.py

@@ -38,7 +38,7 @@ class BuildFrame(wx.Frame):
         if self.constants.update_stage != gui_support.AutoUpdateStages.INACTIVE:
             self.constants.update_stage = gui_support.AutoUpdateStages.BUILDING
 
-        self.Centre()
+        gui_support.Centre(self, self.constants)
         self.frame_modal.ShowWindowModal()
 
         self._invoke_build()

resources/wx_gui/gui_install_oc.py

@@ -33,7 +33,7 @@ class InstallOCFrame(wx.Frame):
         if self.constants.update_stage != gui_support.AutoUpdateStages.INACTIVE:
             self.constants.update_stage = gui_support.AutoUpdateStages.INSTALLING
 
-        self.Centre()
+        gui_support.Centre(self, self.constants)
         self.Show()
 
         self._display_disks()

resources/wx_gui/gui_macos_installer_download.py

@@ -84,7 +84,7 @@ class macOSInstallerDownloadFrame(wx.Frame):
         """
         super(macOSInstallerDownloadFrame, self).__init__(None, title=self.title, size=(300, 200), style=wx.DEFAULT_FRAME_STYLE & ~(wx.RESIZE_BORDER | wx.MAXIMIZE_BOX))
         gui_support.GenerateMenubar(self, self.constants).generate()
-        self.Centre()
+        gui_support.Centre(self, self.constants)
 
         # Title: Pulling installer catalog
         title_label = wx.StaticText(self, label="Pulling installer catalog", pos=(-1,5))

resources/wx_gui/gui_macos_installer_flash.py

@@ -39,7 +39,7 @@ class macOSInstallerFlashFrame(wx.Frame):
 
         self._generate_elements()
 
-        self.Centre()
+        gui_support.Centre(self, self.constants)
         self.Show()
 
         self._populate_installers()

resources/wx_gui/gui_main_menu.py

@@ -39,9 +39,10 @@ class MainFrame(wx.Frame):
 
         self._generate_elements()
 
-        self.Centre()
+        gui_support.Centre(self, self.constants)
         self.Show()
 
+
         self._preflight_checks()
 
 

resources/wx_gui/gui_support.py

@@ -23,6 +23,53 @@ class AutoUpdateStages:
     FINISHED = 5
 
 
+class Centre:
+    """
+    Alternative to wx.Frame.Centre() for macOS Ventura+ on non-Metal GPUs
+    ----------
+    As reported by socamx#3874, all of their non-Metal Mac minis would incorrectly centre
+    at the top of the screen, rather than the screen centre. Half of the window frame would
+    be off-screen, making it rather difficult to grab frame handle and move the window.
+    This bug is only triggered when the application is launched via AppleScript,
+    ie. Relaunch as root for root patching.
+    As calculating screen centre is trivial, this is safe for all non-Metal Macs.
+    Test unit specs:
+    - Macmini4,1 (2010, Nvidia Tesla)
+    - Asus VP228HE 21.5" Display (1920x1080)
+    """
+
+    def __init__(self, frame: wx.Frame, global_constants: constants.Constants) -> None:
+        self.frame: wx.Frame = frame
+        self.constants: constants.Constants = global_constants
+
+        self._centre()
+
+
+    def _centre(self) -> None:
+        """
+        Calculate centre position of screen and set window position
+        """
+        if self.constants.detected_os < os_data.os_data.ventura:
+            self.frame.Centre()
+            return
+
+        if CheckProperties(self.constants).host_is_non_metal() is False:
+            self.frame.Centre()
+            return
+
+        # Get screen resolution
+        screen_resolution = wx.DisplaySize()
+        logging.info(f"Screen resolution: {screen_resolution}")
+        window_size = self.frame.GetSize()
+        logging.info(f"Window size: {window_size}")
+
+        # Calculate window position
+        x_pos = int((screen_resolution[0] - window_size[0]) / 2)
+        y_pos = int((screen_resolution[1] - window_size[1]) / 2)
+
+        self.frame.SetPosition((x_pos, y_pos))
+
+
 class GenerateMenubar:
 
     def __init__(self, frame: wx.Frame, global_constants: constants.Constants) -> None:

resources/wx_gui/gui_sys_patch_start.py

@@ -48,7 +48,7 @@ class SysPatchStartFrame(wx.Frame):
         if not (isinstance(delay, int) | isinstance(delay, float)):
             delay = 1
         time.sleep(delay)
-        self.Centre()
+        gui_support.Centre(self, self.constants)
 
         if self.patches == {}:
             self.patches = sys_patch_detect.DetectRootPatch(self.constants.computer.real_model, self.constants).detect_patch_set()

resources/wx_gui/gui_update.py

@@ -41,7 +41,7 @@ class UpdateFrame(wx.Frame):
             self.parent.Centre()
             self.screen_location = parent.GetScreenPosition()
         else:
-            self.Centre()
+            gui_support.Centre(self, self.constants)
             self.screen_location = self.GetScreenPosition()