From db761ea34cf49be2d0d84fac77b465a1ba15cca8 Mon Sep 17 00:00:00 2001
From: Phillip Carter <phillc54@users.noreply.github.com>
Date: Wed, 6 Mar 2024 15:23:37 +1100
Subject: [PATCH] qtplasmac: fix table view position for all machine
 coordinates

---
 .../screens/qtplasmac/qtplasmac_handler.py    | 30 ++++++-------------
 share/qtvcp/screens/qtplasmac/versions.html   |  7 ++++-
 2 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/share/qtvcp/screens/qtplasmac/qtplasmac_handler.py b/share/qtvcp/screens/qtplasmac/qtplasmac_handler.py
index 6cc993f5e0c..3e5bc9441c2 100644
--- a/share/qtvcp/screens/qtplasmac/qtplasmac_handler.py
+++ b/share/qtvcp/screens/qtplasmac/qtplasmac_handler.py
@@ -1,4 +1,4 @@
-VERSION = '237.308'
+VERSION = '237.309'
 LCNCVER = '2.9'
 DOCSVER = LCNCVER
 
@@ -2072,26 +2072,14 @@ def view_t_pressed(self):
             QApplication.processEvents()
         self.w.gcodegraphics.set_view('Z')
         mid, size = DRAW.extents_info(self.w.gcodegraphics)
-        if self.gcodeProps:
-            mult = 1
-            if self.units == 'in' and self.gcodeProps['gcode_units'] == 'mm':
-                mult = 0.03937
-            elif self.units == 'mm' and self.gcodeProps['gcode_units'] == 'in':
-                mult = 25.4
-            x = (round(float(self.gcodeProps['x'].split()[0]) * mult, 4))
-            y = (round(float(self.gcodeProps['y'].split()[0]) * mult, 4))
-            xl = (round(float(self.gcodeProps['x'].split('=')[1].split()[0]) * mult, 4))
-            yl = (round(float(self.gcodeProps['y'].split('=')[1].split()[0]) * mult, 4))
-        else:
-            x = y = xl = yl = 0
-        if (mid[0] == 0 and mid[1] == 0) or mid[0] > self.xLen or mid[1] > self.yLen or \
-           self.w.view_t.isChecked() or self.w.view_t.isDown() or self.fileClear:
-            mult = 1 if self.units == 'in' else 25.4
-            zoomScale = (self.w.table_zoom_scale.value() * 2)
-            mid = [(self.xLen - (x * 2) - xl) / mult / 2, (self.yLen - (y * 2) - yl) / mult / 2, 0]
-            size = [self.xLen / mult / zoomScale, self.yLen / mult / zoomScale, 0]
-        glTranslatef(-mid[0], -mid[1], -mid[2])
-        self.w.gcodegraphics.set_eyepoint_from_extents(size[0], size[1])
+        mult = 1 if self.units == 'in' else 25.4
+        zoomScale = (self.w.table_zoom_scale.value() * 2)
+        xTableCenter = (self.xMin + self.xLen / 2) / mult - mid[0]
+        yTableCenter = (self.yMin + self.yLen / 2) / mult - mid[1]
+        xSize = self.xLen / mult / zoomScale
+        ySize = self.yLen / mult / zoomScale
+        glTranslatef(-xTableCenter, -yTableCenter, 0)
+        self.w.gcodegraphics.set_eyepoint_from_extents(xSize, ySize)
         self.w.gcodegraphics.perspective = False
         self.w.gcodegraphics.lat = self.w.gcodegraphics.lon = 0
         self.w.gcodegraphics.updateGL()
diff --git a/share/qtvcp/screens/qtplasmac/versions.html b/share/qtvcp/screens/qtplasmac/versions.html
index 942d03982ff..adb765b8da6 100644
--- a/share/qtvcp/screens/qtplasmac/versions.html
+++ b/share/qtvcp/screens/qtplasmac/versions.html
@@ -26,9 +26,14 @@ <h2>QtPlasmaC Version History - LinuxCNC 2.9</h2>
 </table>
 <br>
 <!--- ****** ADD NEXT VERSION BELOW THIS LINE ****** --->
+<br><b><u>237.309 2024 Mar 05</u></b>
+<ul style="margin:0;">
+  <li>fix table view position for all machine coordinates</li>
+</ul>
+
 <br><b><u>237.308 2024 Feb 28</u></b>
 <ul style="margin:0;">
-  <li>fix matl overlay for locales that use a comma as a decimal</li>
+  <li>fix material overlay for locales that use a comma as a decimal</li>
 </ul>
 
 <br><b><u>237.307 2024 Feb 25</u></b>