diff --git a/src/Mod/TechDraw/Gui/QGIView.cpp b/src/Mod/TechDraw/Gui/QGIView.cpp index ca655e8276..6de49b0997 100644 --- a/src/Mod/TechDraw/Gui/QGIView.cpp +++ b/src/Mod/TechDraw/Gui/QGIView.cpp @@ -175,8 +175,7 @@ QVariant QGIView::itemChange(GraphicsItemChange change, const QVariant &value) } else { Gui::ViewProvider *vp = getViewProvider(viewObj); if (vp && !vp->isRestoring()) { - viewObj->setPosition(Rez::appX(newPos.x()), - Rez::appX(isInnerView() ? getYInClip(newPos.y()) : -newPos.y())); + viewObj->setPosition(Rez::appX(newPos.x()), Rez::appX(-newPos.y())); } } @@ -243,14 +242,10 @@ void QGIView::setPosition(qreal xPos, qreal yPos) { // Base::Console().Message("QGIV::setPosition(%.3f, %.3f) (gui)\n", x, y); double newX = xPos; - double newY; + double newY = -yPos; double oldX = pos().x(); double oldY = pos().y(); - if (!isInnerView()) { - newY = -yPos; - } else { - newY = getYInClip(yPos); - } + if (TechDraw::DrawUtil::fpCompare(newX, oldX) && TechDraw::DrawUtil::fpCompare(newY, oldY)) { return; @@ -259,12 +254,6 @@ void QGIView::setPosition(qreal xPos, qreal yPos) } } -//is this needed anymore??? -double QGIView::getYInClip(double y) -{ - return -y; -} - QGIViewClip* QGIView::getClipGroup() { if (!getViewObject()->isInClip()) { diff --git a/src/Mod/TechDraw/Gui/QGIView.h b/src/Mod/TechDraw/Gui/QGIView.h index 4a34602f13..445a18027d 100644 --- a/src/Mod/TechDraw/Gui/QGIView.h +++ b/src/Mod/TechDraw/Gui/QGIView.h @@ -118,7 +118,6 @@ public: inline qreal getY() { return y() * -1; } bool isInnerView() const { return m_innerView; } void isInnerView(bool state) { m_innerView = state; } - double getYInClip(double y); QGIViewClip* getClipGroup();