From ccb6efd6b640ba2067ed9ffa8d64fc6ba0409633 Mon Sep 17 00:00:00 2001
From: Thomas Wilshaw <thomaswilshaw@gmail.com>
Date: Sun, 21 Jun 2020 19:31:40 +0100
Subject: [PATCH] Make gizmos work with translated/zoomed image

Added a new function that inverts the world transform and applies it to
a point. This allows the gizmo code to work in it's own space and not
need to know is the image has been zoomed or translated.
---
 app/widget/viewer/viewerdisplay.cpp | 16 ++++++++++++++--
 app/widget/viewer/viewerdisplay.h   |  2 ++
 2 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/app/widget/viewer/viewerdisplay.cpp b/app/widget/viewer/viewerdisplay.cpp
index c0d3dd858..d89813c1a 100644
--- a/app/widget/viewer/viewerdisplay.cpp
+++ b/app/widget/viewer/viewerdisplay.cpp
@@ -197,10 +197,20 @@ QTransform ViewerDisplayWidget::GenerateWorldTransform()
   return world;
 }
 
+QPoint ViewerDisplayWidget::TransformViewerSpaceToGizmoSpace(QPoint pos)
+{
+  /*
+  * Inversion will only fail is the viewer has been scaled by 0 in any direction
+  * which I think should never happen. Also I don't know what we'd do if it does.
+  */
+  return pos * GenerateWorldTransform().inverted();
+}
+
 void ViewerDisplayWidget::mousePressEvent(QMouseEvent *event)
 {
   if (gizmos_
-      && gizmos_->GizmoPress(gizmo_db_, event->pos(), QVector2D(GetTexturePosition(size())), size())) {
+      && gizmos_->GizmoPress(gizmo_db_, TransformViewerSpaceToGizmoSpace(event->pos()),
+          QVector2D(GetTexturePosition(size())), size())) {
     gizmo_click_ = true;
     gizmo_drag_time_ = GetGizmoTime();
     return;
@@ -246,7 +256,9 @@ void ViewerDisplayWidget::mouseMoveEvent(QMouseEvent *event)
   }
 
   if (gizmo_click_) {
-    gizmos_->GizmoMove(event->pos(), QVector2D(GetTexturePosition(size())), gizmo_drag_time_);
+    gizmos_->GizmoMove(TransformViewerSpaceToGizmoSpace(event->pos()),
+        QVector2D(GetTexturePosition(size())), gizmo_drag_time_);
+    update();
     return;
   }
 
diff --git a/app/widget/viewer/viewerdisplay.h b/app/widget/viewer/viewerdisplay.h
index 7d6a5502e..f81cf71d8 100644
--- a/app/widget/viewer/viewerdisplay.h
+++ b/app/widget/viewer/viewerdisplay.h
@@ -93,6 +93,8 @@ public:
    */
   QTransform GenerateWorldTransform();
 
+  QPoint TransformViewerSpaceToGizmoSpace(QPoint pos);
+
 public slots:
   /**
    * @brief Set the transformation matrix to draw with