First steps in fixing/cleaning up preview/snapper rendering.

[architektonas] / src / actions / actionzoompan.cpp

diff --git a/src/actions/actionzoompan.cpp b/src/actions/actionzoompan.cpp
index 7b8d3a8e552f451a0819b70ab70a53583b7fe6a8..7aa3a42a6903067d8a4d72c70a1a48f0b561b017 100644 (file)
--- a/src/actions/actionzoompan.cpp
+++ b/src/actions/actionzoompan.cpp
@@ -21,6 +21,8 @@
 ActionZoomPan::ActionZoomPan(RS_EntityContainer & container, GraphicView & graphicView):
        ActionInterface("Zoom Pan", container, graphicView)
 {
+       graphicView.snapper.SetVisible(false);
+       graphicView.preview.SetVisible(false);
 }
 
 ActionZoomPan::~ActionZoomPan()
@@ -61,7 +63,7 @@ void ActionZoomPan::mouseMoveEvent(QMouseEvent * e)
 
 void ActionZoomPan::mousePressEvent(QMouseEvent * e)
 {
-       if (e->button() == Qt::MidButton || e->button() == Qt::LeftButton)
+       if (e->button() == Qt::LeftButton || e->button() == Qt::MidButton)
        {
                x1 = e->x();
                y1 = e->y();
@@ -71,12 +73,16 @@ void ActionZoomPan::mousePressEvent(QMouseEvent * e)
 
 void ActionZoomPan::mouseReleaseEvent(QMouseEvent * e)
 {
-       if (e->button() == Qt::RightButton)
+       // Right button? Why?
+       // Is this a regular tool? Is this why? (yup, it is. but lame nonetheless.)
+       // It doesn't revert the cursor correctly if this is not in...
+       // So... We need to fix this... [DONE]
+       /*if (e->button() == Qt::RightButton)
                init(getStatus() - 1);
-       else if (e->button() == Qt::MidButton)
+       else*/ if (e->button() == Qt::MidButton || e->button() == Qt::RightButton)
                init(-1);
        else
-               setStatus(0);
+               setStatus(0);   // Qt::LeftButton
 }
 
 void ActionZoomPan::updateMouseCursor()