Align razor icon with mouse position hotspot in QCursor (qwidget timeline backend). It was offset incorrectly, so the dotted line was not aligned with the actual cursor position, thus cutting in the wrong place.

Author: jonoomph

src/tests/test_timeline_helpers.py

@@ -42,7 +42,7 @@
     sys.path.append(PATH)
 
 from PyQt5.QtCore import QCoreApplication, QPointF, QRectF, Qt
-from PyQt5.QtGui import QColor
+from PyQt5.QtGui import QColor, QCursor
 from PyQt5.QtWidgets import QApplication
 from classes import info
 from classes.updates import UpdateAction
@@ -1804,6 +1804,15 @@ def test_qwidget_cursor_uses_razor_cursor_for_items_when_enabled(self):
         self.assertIs(helper.cursor_value, helper.cursors["razor"])
         self.assertFalse(helper.unset_cursor_called)
 
+    def test_qwidget_razor_cursor_hotspot_matches_web_alignment(self):
+        helper = types.SimpleNamespace()
+
+        cursor = self.qwidget_base_module.TimelineWidgetBase._load_razor_cursor(helper)
+
+        self.assertIsInstance(cursor, QCursor)
+        self.assertEqual(cursor.hotSpot().x(), 0)
+        self.assertEqual(cursor.hotSpot().y(), 2)
+
     def test_qwidget_cursor_keeps_hand_cursor_for_items_when_razor_disabled(self):
         helper = self.make_qwidget_cursor_helper()
         helper.geometry.items = [(QRectF(0.0, 0.0, 100.0, 20.0), object(), False, "clip")]

src/windows/views/timeline_backend/qwidget/base.py

@@ -731,7 +731,9 @@ def _load_razor_cursor(self):
         pixmap = QPixmap(asset_path)
         if pixmap.isNull():
             return QCursor(Qt.CrossCursor)
-        hot_x = min(max(pixmap.width() // 2, 0), max(0, pixmap.width() - 1))
+        # Match the web timeline hotspot, which uses the asset's top-left
+        # corner as the active cursor position.
+        hot_x = 0
         hot_y = min(2, max(0, pixmap.height() - 1))
         return QCursor(pixmap, hot_x, hot_y)