Author: jghali
Date: Sat Mar 12 16:00:07 2022
New Revision: 25019

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=25019
Log:
More removals of deprecated globalPos() calls

Modified:
    trunk/Scribus/scribus/canvas.cpp
    trunk/Scribus/scribus/canvas.h
    trunk/Scribus/scribus/canvasgesture_cellselect.cpp
    trunk/Scribus/scribus/canvasgesture_rectselect.cpp
    trunk/Scribus/scribus/canvasgesture_rectselect.h
    trunk/Scribus/scribus/canvasgesture_resize.cpp
    trunk/Scribus/scribus/canvasgesture_resize.h
    trunk/Scribus/scribus/canvasgesture_rulermove.cpp
    trunk/Scribus/scribus/canvasgesture_rulermove.h
    trunk/Scribus/scribus/canvasmode.cpp
    trunk/Scribus/scribus/canvasmode_edit.cpp
    trunk/Scribus/scribus/canvasmode_edittable.cpp
    trunk/Scribus/scribus/canvasmode_framelinks.cpp
    trunk/Scribus/scribus/canvasmode_measurements.cpp
    trunk/Scribus/scribus/canvasmode_measurements.h
    trunk/Scribus/scribus/canvasmode_nodeedit.cpp
    trunk/Scribus/scribus/canvasmode_normal.cpp
    trunk/Scribus/scribus/canvasmode_rotate.cpp
    trunk/Scribus/scribus/chartableview.cpp
    trunk/Scribus/scribus/ui/hruler.cpp
    trunk/Scribus/scribus/ui/rulermover.cpp
    trunk/Scribus/scribus/ui/rulermover.h
    trunk/Scribus/scribus/ui/storyeditor.cpp

Modified: trunk/Scribus/scribus/canvas.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvas.cpp
==============================================================================
--- trunk/Scribus/scribus/canvas.cpp    (original)
+++ trunk/Scribus/scribus/canvas.cpp    Sat Mar 12 16:00:07 2022
@@ -119,7 +119,8 @@
 // (xoff, yoff) = ruler origin relative to canvas top-left
        double xoff = m_doc->rulerXoffset - xmin;
        double yoff = m_doc->rulerYoffset - ymin;
-       if (m_doc->guidesPrefs().rulerMode) {
+       if (m_doc->guidesPrefs().rulerMode)
+       {
                xoff += m_doc->currentPage()->xOffset();
                yoff += m_doc->currentPage()->yOffset();
        }
@@ -131,8 +132,8 @@
 
 // number of ruler divisions from ruler origin to ruler mark closest to the
 // selected mouse coordinate
-       double xn = qRound((p.x()/sc - xoff)/xsp);
-       double yn = qRound((p.y()/sc - yoff)/ysp);
+       double xn = qRound((p.x() / sc - xoff) / xsp);
+       double yn = qRound((p.y() / sc - yoff) / ysp);
 
 // xn*xsp + xoff, yn*ysp + yoff = distance in canvas coordinates
 // from canvas top-left to ruler mark closest to selected mouse coordinate
@@ -140,9 +141,9 @@
 // If these round to the selected mouse coordinate, use these to define
 // (x,y), the coordinates of the desired point relative to the canvas top-left.
 // Otherwise, simply scale the mouse coordinates.
-       double x = ( qRound(sc*(xn*xsp + xoff)) == p.x() ? 
+       double x = ( qRound(sc*(xn * xsp + xoff)) == p.x() ? 
                     xn * xsp + xoff : p.x() / sc );
-       double y = ( qRound(sc*(yn*ysp + yoff)) == p.y() ? 
+       double y = ( qRound(sc*(yn * ysp + yoff)) == p.y() ? 
                     yn * ysp + yoff : p.y() / sc );
                
 // Finally, add xmin,ymin to get the absolute canvas coordinates of the
@@ -150,14 +151,53 @@
        return FPoint(x + xmin, y + ymin);
 }
 
-
-/*
 FPoint Canvas::localToCanvas(QPointF p) const
 {
-       return FPoint(p.x() / m_viewMode.scale + m_doc->minCanvasCoordinate.x() 
, 
-                                 p.y() / m_viewMode.scale + 
m_doc->minCanvasCoordinate.y());   
-}
-*/
+
+       /* Allow the user to select the exact coordinate represented by a ruler 
mark
+          when the mouse is lined up with the ruler, rather than returning the
+          coordinate represented by the mathematical centre of the pixel which
+          may not be exactly the same as the coordinate represented by the 
ruler.
+       */
+
+       // (xmin, xmax) = canvas top-left
+       double xmin = m_doc->minCanvasCoordinate.x();
+       double ymin = m_doc->minCanvasCoordinate.y();
+
+       // (xoff, yoff) = ruler origin relative to canvas top-left
+       double xoff = m_doc->rulerXoffset - xmin;
+       double yoff = m_doc->rulerYoffset - ymin;
+       if (m_doc->guidesPrefs().rulerMode)
+       {
+               xoff += m_doc->currentPage()->xOffset();
+               yoff += m_doc->currentPage()->yOffset();
+       }
+       // (xsp, ysp) = spacing of ruler divisions
+       double xsp = m_doc->view()->horizRuler->ruleSpacing();
+       double ysp = m_doc->view()->vertRuler->ruleSpacing();
+
+       double sc = m_viewMode.scale;
+
+       // number of ruler divisions from ruler origin to ruler mark closest to 
the
+       // selected mouse coordinate
+       double xn = qRound((p.x() / sc - xoff) / xsp);
+       double yn = qRound((p.y() / sc - yoff) / ysp);
+
+       // xn*xsp + xoff, yn*ysp + yoff = distance in canvas coordinates
+       // from canvas top-left to ruler mark closest to selected mouse 
coordinate
+
+       // If these round to the selected mouse coordinate, use these to define
+       // (x,y), the coordinates of the desired point relative to the canvas 
top-left.
+       // Otherwise, simply scale the mouse coordinates.
+       double x = (qRound(sc * (xn * xsp + xoff)) == p.x() ?
+               xn * xsp + xoff : p.x() / sc);
+       double y = (qRound(sc * (yn * ysp + yoff)) == p.y() ?
+               yn * ysp + yoff : p.y() / sc);
+
+       // Finally, add xmin,ymin to get the absolute canvas coordinates of the
+       // desired point.
+       return FPoint(x + xmin, y + ymin);
+}
 
 QPoint Canvas::canvasToLocal(const FPoint& p) const
 {
@@ -553,18 +593,18 @@
        return nullptr;
 }
 
-bool Canvas::cursorOverTextFrameControl(QPoint globalPos, PageItem* frame)
-{
-       FPoint mp=globalToCanvas(globalPos);
+bool Canvas::cursorOverTextFrameControl(QPointF globalPos, PageItem* frame)
+{
+       FPoint mp = globalToCanvas(globalPos);
        qreal sideLength = 10 / qMax(m_viewMode.scale, 1.0);
        qreal left  = frame->xPos() + frame->width() - sideLength;// / 2;
        qreal right = left + sideLength;
        qreal top   = frame->yPos() + frame->height() - sideLength;// * 1.5;
        qreal bottom = top + sideLength;
-       return mp.x()>left && mp.x()<right && mp.y()>top && mp.y()<bottom;
-}
-
-bool Canvas::cursorOverFrameControl(QPoint globalPos, const QRectF& 
targetRect, PageItem* frame)
+       return mp.x() > left && mp.x() < right && mp.y() > top && mp.y() < 
bottom;
+}
+
+bool Canvas::cursorOverFrameControl(QPointF globalPos, const QRectF& 
targetRect, PageItem* frame)
 {
        FPoint mp = globalToCanvas(globalPos);
        QRectF tg = targetRect.translated(frame->xPos(), frame->yPos());

Modified: trunk/Scribus/scribus/canvas.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvas.h
==============================================================================
--- trunk/Scribus/scribus/canvas.h      (original)
+++ trunk/Scribus/scribus/canvas.h      Sat Mar 12 16:00:07 2022
@@ -157,7 +157,7 @@
        QRect canvasToGlobal(const QRectF& p) const;
        QRectF canvasToGlobalF(const QRectF& p) const;
        FPoint localToCanvas(QPoint p) const;
-//     FPoint localToCanvas(QPointF p) const;
+       FPoint localToCanvas(QPointF p) const;
        FPoint globalToCanvas(QPoint p) const;
        FPoint globalToCanvas(QPointF p) const;
        QRectF globalToCanvas(QRect p) const;
@@ -168,8 +168,8 @@
        bool hitsCanvasPoint(QPointF globalPoint, QPointF canvasPoint) const;
        bool hitsCanvasPoint(const FPoint& globalPoint, const QPointF& 
canvasPoint) const;
        QRect exposedRect() const;
-       bool cursorOverTextFrameControl(QPoint globalPos, PageItem* frame);
-       bool cursorOverFrameControl(QPoint globalPos, const QRectF& targetRect, 
PageItem* frame);
+       bool cursorOverTextFrameControl(QPointF globalPos, PageItem* frame);
+       bool cursorOverFrameControl(QPointF globalPos, const QRectF& 
targetRect, PageItem* frame);
        /** Returns the framehandle or INSIDE if the position falls into the 
frame. */
        FrameHandle frameHitTest(QPointF canvasPoint, PageItem* item) const;
        FrameHandle frameHitTest(QPointF canvasPoint, const QRectF& frame) 
const;

Modified: trunk/Scribus/scribus/canvasgesture_cellselect.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_cellselect.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_cellselect.cpp  (original)
+++ trunk/Scribus/scribus/canvasgesture_cellselect.cpp  Sat Mar 12 16:00:07 2022
@@ -56,7 +56,7 @@
        {
                // Show the table popup menu.
                m_view->setCursor(Qt::ArrowCursor);
-               m_view->m_ScMW->scrMenuMgr->runMenuAtPos("ItemTable", 
event->globalPos());
+               m_view->m_ScMW->scrMenuMgr->runMenuAtPos("ItemTable", 
event->globalPosition().toPoint());
        }
        else
        {

Modified: trunk/Scribus/scribus/canvasgesture_rectselect.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_rectselect.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_rectselect.cpp  (original)
+++ trunk/Scribus/scribus/canvasgesture_rectselect.cpp  Sat Mar 12 16:00:07 2022
@@ -36,20 +36,21 @@
 void RectSelect::leaveEvent(QEvent * e){}
 
 
-void RectSelect::prepare(QPoint globalStartPos)
+void RectSelect::prepare(QPointF globalStartPos)
 {
        if (!m_selectionRubberBand)
                m_selectionRubberBand = new 
SelectionRubberBand(QRubberBand::Rectangle, m_view);
        setStart(globalStartPos);
 //FIXME Move to new code like SelectionRubberBand
+       QPoint selectionPoint = m_view->mapFromGlobal(globalStartPos).toPoint();
        m_selectionRubberBand->setWindowOpacity(0.5);
-       
m_selectionRubberBand->setGeometry(QRect(m_view->mapFromGlobal(globalStartPos), 
m_view->mapFromGlobal(globalStartPos)));
+       m_selectionRubberBand->setGeometry(QRect(selectionPoint, 
selectionPoint));
 }
 
 void RectSelect::clear()
 {
        m_selectionRubberBand->hide();
-       m_start = QPoint(0,0);
+       m_start = QPointF(0,0);
 }
 
 
@@ -66,14 +67,16 @@
        CanvasGesture::deactivate(fromGesture);
 }
 
-void RectSelect::setStart(QPoint globalPos)
+void RectSelect::setStart(QPointF globalPos)
 {
        m_start = globalPos;
 }
 
-void RectSelect::setEnd(QPoint globalPos)
+void RectSelect::setEnd(QPointF globalPos)
 {
-       
m_selectionRubberBand->setGeometry(QRect(m_view->mapFromGlobal(m_start), 
m_view->mapFromGlobal(globalPos)).normalized());
+       QPointF startPoint = m_view->mapFromGlobal(m_start);
+       QPointF endPoint = m_view->mapFromGlobal(globalPos);
+       m_selectionRubberBand->setGeometry(QRect(startPoint.toPoint(), 
endPoint.toPoint()).normalized());
 }
 
 
@@ -86,22 +89,22 @@
 
 void RectSelect::mousePressEvent(QMouseEvent *m)
 {
-       prepare(m->globalPos());
+       prepare(m->globalPosition());
        m->accept();
 }
 
 
 void RectSelect::mouseReleaseEvent(QMouseEvent *m)
 {
-//     qDebug() << "RectSelect::mouseRelease" << m->globalPos();
-       setEnd(m->globalPos());
+//     qDebug() << "RectSelect::mouseRelease" << m->globalPosition();
+       setEnd(m->globalPosition());
        m->accept();
        m_view->stopGesture();
 }
 
 void RectSelect::mouseMoveEvent(QMouseEvent *m)
 {
-       setEnd(m->globalPos());
+       setEnd(m->globalPosition());
        m->accept();
 }
 

Modified: trunk/Scribus/scribus/canvasgesture_rectselect.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_rectselect.h
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_rectselect.h    (original)
+++ trunk/Scribus/scribus/canvasgesture_rectselect.h    Sat Mar 12 16:00:07 2022
@@ -51,7 +51,7 @@
                explicit RectSelect (CanvasMode* parent) : 
CanvasGesture(parent), m_start(0,0), m_selectionRubberBand(nullptr) {};
                ~RectSelect() override = default;
 
-               void prepare(QPoint globalStartPos);
+               void prepare(QPointF globalStartPos);
                void clear();
 
                void enterEvent(QEvent * e) override;
@@ -69,10 +69,10 @@
                QRectF result() const ;
 
        private:
-               void setStart(QPoint globalPos);
-               void setEnd(QPoint globalPos);
+               void setStart(QPointF globalPos);
+               void setEnd(QPointF globalPos);
 
-               QPoint m_start;
+               QPointF m_start;
                SelectionRubberBand* m_selectionRubberBand;
 };
 

Modified: trunk/Scribus/scribus/canvasgesture_resize.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_resize.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_resize.cpp      (original)
+++ trunk/Scribus/scribus/canvasgesture_resize.cpp      Sat Mar 12 16:00:07 2022
@@ -217,7 +217,7 @@
                //add action itemResizeToMargin for right click on item`s 
handlers
                else if (m->button() == Qt::RightButton)
                {
-                       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPos());
+                       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
                        Canvas::FrameHandle fh = 
m_canvas->frameHitTest(QPointF(mousePointDoc.x(),mousePointDoc.y()), currItem);
                        m_doc->itemResizeToMargin(currItem, fh);
                }
@@ -441,9 +441,9 @@
                doResize(m->modifiers() & Qt::AltModifier);
        }
        m->accept();
-       QPoint point = m->globalPos() - (m_canvas->mapToParent(QPoint(0, 0)) + 
m_canvas->parentWidget()->mapToGlobal(QPoint(0, 0)));
-       m_view->ensureVisible(point.x(), point.y(), 20, 20);
-       m_canvas->repaint();
+       QPointF point = m->globalPosition() - (m_canvas->mapToParent(QPointF(0, 
0)) + m_canvas->parentWidget()->mapToGlobal(QPointF(0, 0)));
+       m_view->ensureVisible(qRound(point.x()), qRound(point.y()), 20, 20);
+       m_canvas->update();
        QRectF newBounds = m_bounds.normalized();
        m_canvas->displaySizeHUD(m->globalPosition(), newBounds.width() - 
m_extraWidth, newBounds.height() - m_extraHeight);
 }
@@ -453,7 +453,7 @@
 void ResizeGesture::adjustBounds(QMouseEvent *m)
 {
        QTransform rotation;
-       FPoint docPoint = m_canvas->globalToCanvas(m->globalPos());
+       FPoint docPoint = m_canvas->globalToCanvas(m->globalPosition());
 
        // proportional resize
        bool constrainRatio = ((m->modifiers() & Qt::ControlModifier) != 
Qt::NoModifier);
@@ -462,7 +462,7 @@
        bool centerPivot = ((m->modifiers() & Qt::ShiftModifier) != 
Qt::NoModifier);
 
 /*
-       if (m_mousePressPoint == m->globalPos())
+       if (m_mousePressPoint == m->globalPosition())
        {
                m_bounds = m_mousePressBounds;
                return;
@@ -493,7 +493,7 @@
                        docPoint = FPoint(x, docPoint.y());
                else 
                        docPoint = FPoint(x,y);
-//             qDebug() << "resize snap grid/guides:" << m->globalPos() << 
"-->" << m_canvas->canvasToGlobal(docPoint);
+//             qDebug() << "resize snap grid/guides:" << m->globalPosition() 
<< "-->" << m_canvas->canvasToGlobal(docPoint);
        }
        
        // un-rotate point
@@ -948,8 +948,8 @@
 
 void ResizeGesture::mousePressEvent(QMouseEvent *m)
 {
-       FPoint point = m_canvas->globalToCanvas(m->globalPos());
-       m_mousePressPoint = m->globalPos();
+       FPoint point = m_canvas->globalToCanvas(m->globalPosition());
+       m_mousePressPoint = m->globalPosition();
        if (m_doc->m_Selection->isEmpty())
        {
                m_handle = Canvas::OUTSIDE;

Modified: trunk/Scribus/scribus/canvasgesture_resize.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_resize.h
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_resize.h        (original)
+++ trunk/Scribus/scribus/canvasgesture_resize.h        Sat Mar 12 16:00:07 2022
@@ -78,7 +78,7 @@
                FPoint applyGuides(const FPoint& docPoint);
 
                Canvas::FrameHandle m_handle {Canvas::INSIDE};
-               QPoint m_mousePressPoint;
+               QPointF m_mousePressPoint;
                QRectF m_bounds;
                QRectF m_mousePressBounds;
                QRectF m_origBounds;

Modified: trunk/Scribus/scribus/canvasgesture_rulermove.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_rulermove.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_rulermove.cpp   (original)
+++ trunk/Scribus/scribus/canvasgesture_rulermove.cpp   Sat Mar 12 16:00:07 2022
@@ -44,29 +44,30 @@
                page = m_doc->OnPage(m_mousePoint.x(), m_mousePoint.y());
        if (page == -1)
                return;
-       ScPage* dragToPage=m_doc->Pages->at(page);
+       ScPage* dragToPage = m_doc->Pages->at(page);
        if (!dragToPage)
                return;
        if (m_haveGuide)
                dragToPage = m_doc->Pages->at(m_page);
        QColor color(m_doc->guidesPrefs().guideColor);
        p->save();
+       QPoint roundedXY = m_xy.toPoint();
        QPoint pageOrigin = 
m_canvas->canvasToLocal(QPointF(dragToPage->xOffset(), dragToPage->yOffset()));
        QSize pageSize = (QSizeF(dragToPage->width(), dragToPage->height()) * 
m_canvas->scale()).toSize();
        switch (m_mode)
        {
                case HORIZONTAL:
                        p->setPen(QPen(color, 1.0, Qt::DashDotLine, 
Qt::FlatCap, Qt::MiterJoin));
-                       p->drawLine(QPoint(pageOrigin.x(), m_xy.y()), 
QPoint(pageOrigin.x() + pageSize.width(), m_xy.y()));
+                       p->drawLine(QPoint(pageOrigin.x(), roundedXY.y()), 
QPoint(pageOrigin.x() + pageSize.width(), roundedXY.y()));
                        break;
                case VERTICAL:
                        p->setPen(QPen(color, 1.0, Qt::DashDotLine, 
Qt::FlatCap, Qt::MiterJoin));
-                       p->drawLine(QPoint(m_xy.x(), pageOrigin.y()), 
QPoint(m_xy.x(), pageOrigin.y() + pageSize.height()));
+                       p->drawLine(QPoint(roundedXY.x(), pageOrigin.y()), 
QPoint(roundedXY.x(), pageOrigin.y() + pageSize.height()));
                        break;
                case ORIGIN:
                        p->setPen(QPen(color, 1.0, Qt::DotLine, Qt::FlatCap, 
Qt::MiterJoin));
-                       p->drawLine(QPoint(m_xy.x(), 0), QPoint(m_xy.x(), 
m_canvas->height()));
-                       p->drawLine(QPoint(0, m_xy.y()), 
QPoint(m_canvas->width(), m_xy.y()));
+                       p->drawLine(QPoint(roundedXY.x(), 0), 
QPoint(roundedXY.x(), m_canvas->height()));
+                       p->drawLine(QPoint(0, roundedXY.y()), 
QPoint(m_canvas->width(), roundedXY.y()));
                        break;
        }
        p->restore();
@@ -87,11 +88,11 @@
 {
        CanvasGesture::activate(fromGesture);
 
-       m_haveCursor = (qApp->overrideCursor() != nullptr);
-       if ( (!fromGesture) && qApp->overrideCursor())
+       m_haveCursor = (QApplication::overrideCursor() != nullptr);
+       if ( (!fromGesture) && QApplication::overrideCursor())
        {
                m_haveCursor = true;
-               m_cursor = *(qApp->overrideCursor());
+               m_cursor = *(QApplication::overrideCursor());
        }
        else
        {
@@ -100,13 +101,13 @@
        switch (m_mode)
        {
                case HORIZONTAL:
-                       qApp->changeOverrideCursor(QCursor(Qt::SplitVCursor));
+                       
QApplication::changeOverrideCursor(QCursor(Qt::SplitVCursor));
                        break;
                case VERTICAL:
-                       qApp->changeOverrideCursor(QCursor(Qt::SplitHCursor));
+                       
QApplication::changeOverrideCursor(QCursor(Qt::SplitHCursor));
                        break;
                case ORIGIN:
-                       qApp->changeOverrideCursor(QCursor(Qt::CrossCursor));
+                       
QApplication::changeOverrideCursor(QCursor(Qt::CrossCursor));
                        break;
        }
        emit guideInfo(m_mode, m_guide);
@@ -115,7 +116,7 @@
 void RulerGesture::deactivate(bool fromGesture)
 {
        if (m_haveCursor)
-               qApp->changeOverrideCursor(m_cursor);
+               QApplication::changeOverrideCursor(m_cursor);
        m_haveGuide = false;
        m_mousePoint = FPoint();
 
@@ -156,7 +157,7 @@
 
 void RulerGesture::mouseSelectGuide(QMouseEvent *m)
 {
-       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPosition());
        const int page = m_doc->OnPage(mousePointDoc.x(), mousePointDoc.y());
        if ((m_doc->guidesPrefs().guidesShown) && page >= 0)
        {
@@ -178,10 +179,10 @@
 
 void RulerGesture::movePoint(QMouseEvent* m, bool mouseRelease)
 {
-       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPosition());
        const int page = m_doc->OnPage(mousePointDoc.x(), mousePointDoc.y());
-       QRect  viewport(m_view->viewport()->mapToGlobal(QPoint(0,0)), 
QSize(m_view->visibleWidth(), m_view->visibleHeight()));
-       QPoint newMousePoint = m->globalPos() - 
(m_canvas->mapToParent(QPoint(0, 0)) + 
m_canvas->parentWidget()->mapToGlobal(QPoint(0, 0)));
+       QRectF  viewport(m_view->viewport()->mapToGlobal(QPointF(0, 0)), 
QSizeF(m_view->visibleWidth(), m_view->visibleHeight()));
+       QPointF newMousePoint = m->globalPosition() - 
(m_canvas->mapToParent(QPointF(0, 0)) + 
m_canvas->parentWidget()->mapToGlobal(QPointF(0, 0)));
        double x = mousePointDoc.x();
        double y = mousePointDoc.y();
        switch (m_mode)
@@ -193,9 +194,9 @@
                case HORIZONTAL:
                        if (!m_ScMW->doc->guidesPrefs().guidesShown)
                                break;
-                       m_canvas->update(0, m_xy.y() - 2, m_canvas->width(), 4);
+                       m_canvas->update(0, qRound(m_xy.y()) - 2, 
m_canvas->width(), 4);
                        
-                       if ((page >= 0) && (viewport.contains(m->globalPos())))
+                       if ((page >= 0) && 
(viewport.contains(m->globalPosition())))
                        {
                                ScPage* currentPage = m_doc->Pages->at(page);
                                if (m_doc->SnapElement)
@@ -211,7 +212,7 @@
                                }
                                if (!m_haveGuide)
                                {
-                                       
qApp->changeOverrideCursor(QCursor(Qt::SplitVCursor));
+                                       
QApplication::changeOverrideCursor(QCursor(Qt::SplitVCursor));
                                        if (mouseRelease)
                                        {
                                                
currentPage->guides.addHorizontal(y - currentPage->yOffset(), 
GuideManagerCore::Standard);
@@ -237,9 +238,9 @@
                                }
                                else
                                {
-                                       QCursor* cursor = 
qApp->overrideCursor();
+                                       QCursor* cursor = 
QApplication::overrideCursor();
                                        if (cursor && (cursor->shape() != 
Qt::SplitVCursor))
-                                               
qApp->changeOverrideCursor(QCursor(Qt::SplitVCursor));
+                                               
QApplication::changeOverrideCursor(QCursor(Qt::SplitVCursor));
                                }
                                m_currentGuide = y - currentPage->yOffset();
                        }
@@ -247,7 +248,7 @@
                        { 
                                if (m_haveGuide)
                                {
-                                       
qApp->changeOverrideCursor(IconManager::instance().loadCursor("DelPoint.png"));
+                                       
QApplication::changeOverrideCursor(IconManager::instance().loadCursor("DelPoint.png"));
                                        if (mouseRelease)
                                        {
                                                
m_doc->Pages->at(m_page)->guides.deleteHorizontal( m_guide, 
GuideManagerCore::Standard);
@@ -262,8 +263,8 @@
                case VERTICAL:
                        if (!m_ScMW->doc->guidesPrefs().guidesShown)
                                break;
-                       m_canvas->update(m_xy.x() - 2, 0, 4, 
m_canvas->height());
-                       if ((page >= 0) && viewport.contains(m->globalPos()))
+                       m_canvas->update(qRound(m_xy.x()) - 2, 0, 4, 
m_canvas->height());
+                       if ((page >= 0) && 
viewport.contains(m->globalPosition()))
                        {
                                ScPage* currentPage = m_doc->Pages->at(page);
                                if (m_doc->SnapElement)
@@ -279,7 +280,7 @@
                                }
                                if (!m_haveGuide)
                                {
-                                       
qApp->changeOverrideCursor(QCursor(Qt::SplitHCursor));
+                                       
QApplication::changeOverrideCursor(QCursor(Qt::SplitHCursor));
                                        if (mouseRelease)
                                        {
                                                
currentPage->guides.addVertical(x - currentPage->xOffset(), 
GuideManagerCore::Standard);
@@ -305,9 +306,9 @@
                                }
                                else
                                {
-                                       QCursor* cursor = 
qApp->overrideCursor();
+                                       QCursor* cursor = 
QApplication::overrideCursor();
                                        if (cursor && (cursor->shape() != 
Qt::SplitHCursor))
-                                               
qApp->changeOverrideCursor(QCursor(Qt::SplitHCursor));
+                                               
QApplication::changeOverrideCursor(QCursor(Qt::SplitHCursor));
                                }
                                m_currentGuide = x - currentPage->xOffset();
                        }
@@ -315,7 +316,7 @@
                        { 
                                if (m_haveGuide)
                                {
-                                       
qApp->changeOverrideCursor(IconManager::instance().loadCursor("DelPoint.png"));
+                                       
QApplication::changeOverrideCursor(IconManager::instance().loadCursor("DelPoint.png"));
                                        if (mouseRelease)
                                        {
                                                
m_doc->Pages->at(m_page)->guides.deleteVertical( m_guide, 
GuideManagerCore::Standard);
@@ -346,7 +347,7 @@
 
 void RulerGesture::mouseMoveEvent(QMouseEvent* m)
 {
-       m_mousePoint=m_canvas->globalToCanvas(m->globalPos());
+       m_mousePoint = m_canvas->globalToCanvas(m->globalPosition());
        m->accept();
        if (m_view->moveTimerElapsed())
        {
@@ -386,12 +387,13 @@
 void RulerGesture::mousePressEvent(QMouseEvent* m)
 {
        m->accept();
-       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
-       m_view->registerMousePress(m->globalPos());
+
+       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPosition());
+       m_view->registerMousePress(m->globalPosition());
+
        if (mouseHitsGuide(mousePointDoc))
-       {
-               m_xy = m->globalPos() - (m_canvas->mapToParent(QPoint(0, 0)) + 
m_canvas->parentWidget()->mapToGlobal(QPoint(0, 0)));
-       }
+               m_xy = m->globalPosition() - (m_canvas->mapToParent(QPointF(0, 
0)) + m_canvas->parentWidget()->mapToGlobal(QPointF(0, 0)));
+
        if (m_ScMW->doc->guidesPrefs().guidesShown)
                emit guideInfo(m_mode, m_currentGuide);
 }

Modified: trunk/Scribus/scribus/canvasgesture_rulermove.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasgesture_rulermove.h
==============================================================================
--- trunk/Scribus/scribus/canvasgesture_rulermove.h     (original)
+++ trunk/Scribus/scribus/canvasgesture_rulermove.h     Sat Mar 12 16:00:07 2022
@@ -85,7 +85,7 @@
                FPoint m_mousePoint;
                Mode m_mode;
                QCursor m_cursor;
-               QPoint m_xy;
+               QPointF m_xy;
                ScribusMainWindow* m_ScMW {nullptr};
                bool m_haveCursor {false};
                bool m_haveGuide {false};

Modified: trunk/Scribus/scribus/canvasmode.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode.cpp        (original)
+++ trunk/Scribus/scribus/canvasmode.cpp        Sat Mar 12 16:00:07 2022
@@ -833,7 +833,7 @@
 
 bool CanvasMode::commonMouseMove(QMouseEvent *m)
 {
-       //m_mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       //m_mousePointDoc = m_canvas->globalToCanvas(m->globalPosition());
        if ((m_canvas->m_viewMode.m_MouseButtonPressed && (m->buttons() & 
Qt::RightButton) && (m->modifiers() & Qt::ControlModifier)) || 
((!(m->modifiers() & Qt::ControlModifier)) && (m->buttons() & 
Qt::MiddleButton)))
        {
                if (!m_panGesture)
@@ -977,7 +977,7 @@
                        ContextMenu* cmen=nullptr;
                        if (m_doc->m_Selection->isEmpty())
                        {
-                               //CB We should be able to get this calculated 
by the canvas.... it is already in m_canvas->globalToCanvas(m->globalPos());
+                               //CB We should be able to get this calculated 
by the canvas.... it is already in 
m_canvas->globalToCanvas(m->globalPosition());
                                QPoint p(QCursor::pos() - 
mainWindow->mapToGlobal(QPoint(0,0)));
                                FPoint fp(p.x() / m_view->scale() + 
m_doc->minCanvasCoordinate.x(),
                                p.y() / m_view->scale() + 
m_doc->minCanvasCoordinate.y());
@@ -1142,7 +1142,7 @@
                        m_view->setCursor(QCursor(Qt::ArrowCursor));
                        if (m_doc->m_Selection->isEmpty())
                        {
-                               //CB We should be able to get this calculated 
by the canvas.... it is already in m_canvas->globalToCanvas(m->globalPos());
+                               //CB We should be able to get this calculated 
by the canvas.... it is already in 
m_canvas->globalToCanvas(m->globalPosition());
                                FPoint fp = 
m_canvas->globalToCanvas(QCursor::pos());
                                cmen = new ContextMenu(mainWindow, m_doc, 
fp.x(), fp.y());
                        }

Modified: trunk/Scribus/scribus/canvasmode_edit.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_edit.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_edit.cpp   (original)
+++ trunk/Scribus/scribus/canvasmode_edit.cpp   Sat Mar 12 16:00:07 2022
@@ -166,7 +166,7 @@
                QPoint oldP = m_canvas->canvasToLocal(QPointF(mRulerGuide, 
itPos.y()));
                mRulerGuide = base + xp;
                QPoint p = m_canvas->canvasToLocal(QPointF(mRulerGuide, 
itPos.y() + currItem->height() * mm.m22()));
-               m_canvas->update(QRect(oldP.x()-2, oldP.y(), p.x()+2, p.y()));
+               m_canvas->update(QRect(oldP.x() - 2, oldP.y(), p.x() + 2, 
p.y()));
        }
 }
 
@@ -392,7 +392,8 @@
 
 void CanvasMode_Edit::mouseMoveEvent(QMouseEvent *m)
 {
-       const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       const QPoint globalPos = m->globalPosition().toPoint();
+       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
        
        double newX, newY;
        PageItem *currItem;
@@ -438,7 +439,7 @@
                                int 
refEndSel(currItem->asTextFrame()->itemText.endOfSelection());
                                currItem->itemText.deselectAll();
                                currItem->HasSel = false;
-                               m_view->slotSetCurs(m->globalPos().x(), 
m->globalPos().y());
+                               m_view->slotSetCurs(globalPos.x(), 
globalPos.y());
                                //Make sure we don't go here if the old cursor 
position was not set
                                if (oldCp!=-1 && currItem->itemText.length() > 
0)
                                {
@@ -513,7 +514,7 @@
                        SeRx = newX;
                        SeRy = newY;
                        QPoint startP = m_canvas->canvasToGlobal(QPointF(Mxp, 
Myp));
-                       
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(m->globalPos())).normalized());
+                       
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(globalPos)).normalized());
                        m_view->setRedrawMarkerShown(true);
                        m_view->HaveSelRect = true;
                        return;
@@ -537,19 +538,18 @@
                                ss->set("ETEA", QString(""));
                }
        }
-       const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
-
-       bool inText;
-       PageItem *currItem;
+
+       const QPoint globalPos = m->globalPosition().toPoint();
+       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
+       
        m_canvas->PaintSizeRect(QRect());
-       QTransform pm;
        m_canvas->m_viewMode.m_MouseButtonPressed = true;
        m_canvas->m_viewMode.operItemMoving = false;
        m_view->HaveSelRect = false;
        m_doc->DragP = false;
        m_doc->leaveDrag = false;
        m->accept();
-       m_view->registerMousePress(m->globalPos());
+       m_view->registerMousePress(m->globalPosition());
        Mxp = mousePointDoc.x();
        Myp = mousePointDoc.y();
        SeRx = Mxp;
@@ -563,13 +563,15 @@
        }
 
        frameResizeHandle = 0;
-       int oldP=0;
+       int oldP = 0;
+
+       PageItem* currItem{ nullptr };
        if (GetItem(&currItem))
        {
 //             m_view->slotDoCurs(false);
                if ((!currItem->locked() || currItem->isTextFrame()) && 
!currItem->isLine())
                {
-                       FPoint canvasPoint = 
m_canvas->globalToCanvas(m->globalPos());
+                       FPoint canvasPoint = 
m_canvas->globalToCanvas(m->globalPosition());
                        if (m_canvas->frameHitTest(QPointF(canvasPoint.x(), 
canvasPoint.y()), currItem) < 0)
                        {
                                m_doc->m_Selection->delaySignalsOn();
@@ -590,7 +592,7 @@
                                        }
                                        if (currItem->isTextFrame())
                                        {
-                                               
m_view->slotSetCurs(m->globalPos().x(), m->globalPos().y());
+                                               
m_view->slotSetCurs(globalPos.x(), globalPos.y());
                                                oldCp = 
currItem->itemText.cursorPosition();
                                        }
                                }
@@ -612,7 +614,7 @@
                //CB Where we set the cursor for a click in text frame
                if (currItem->isTextFrame())
                {
-                       inText = m_view->slotSetCurs(m->globalPos().x(), 
m->globalPos().y());
+                       bool inText = m_view->slotSetCurs(globalPos.x(), 
globalPos.y());
                        //CB If we clicked outside a text frame to go out of 
edit mode and deselect the frame
                        if (!inText)
                        {
@@ -728,8 +730,8 @@
 #ifdef GESTURE_FRAME_PREVIEW
        clearPixmapCache();
 #endif // GESTURE_FRAME_PREVIEW
-       const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
-       PageItem *currItem;
+       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
+       PageItem *currItem = nullptr;
        m_canvas->m_viewMode.m_MouseButtonPressed = false;
        m_canvas->resetRenderMode();
        m->accept();
@@ -941,20 +943,18 @@
        const unsigned SELECT_IN_GROUP = Qt::AltModifier;
        const unsigned SELECT_MULTIPLE = Qt::ShiftModifier;
        const unsigned SELECT_BENEATH = Qt::ControlModifier;
-       QTransform p;
-       QRectF mpo;
-       PageItem *currItem;
+
        m_canvas->m_viewMode.m_MouseButtonPressed = true;
-       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
-       Mxp = mousePointDoc.x(); //m->x()/m_canvas->scale());
-       Myp = mousePointDoc.y(); //m->y()/m_canvas->scale());
+
+       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPosition());
+       Mxp = mousePointDoc.x();
+       Myp = mousePointDoc.y();
        double grabRadius = m_doc->guidesPrefs().grabRadius / m_canvas->scale();
-       int MxpS = static_cast<int>(mousePointDoc.x()); 
//m->x()/m_canvas->scale() + 0*m_doc->minCanvasCoordinate.x());
-       int MypS = static_cast<int>(mousePointDoc.y()); 
//m->y()/m_canvas->scale() + 0*m_doc->minCanvasCoordinate.y());
-       mpo = QRectF(Mxp-grabRadius, Myp-grabRadius, grabRadius*2, 
grabRadius*2);
-//     mpo.translate(m_doc->minCanvasCoordinate.x() * m_canvas->scale(), 
m_doc->minCanvasCoordinate.y() * m_canvas->scale());
+       int MxpS = static_cast<int>(mousePointDoc.x());
+       int MypS = static_cast<int>(mousePointDoc.y());
+       QRectF mpo(Mxp - grabRadius, Myp - grabRadius, grabRadius * 2, 
grabRadius * 2);
+
        m_doc->nodeEdit.deselect();
-//     int a;
        if (!m_doc->masterPageMode())
        {
                int pgNum = -1;
@@ -1004,7 +1004,7 @@
                m_view->setRulerPos(m_view->contentsX(), m_view->contentsY());
        }
        
-       currItem = nullptr;
+       PageItem* currItem = nullptr;
        if ((m->modifiers() & SELECT_BENEATH) != 0)
        {
                for (int i = 0; i < m_doc->m_Selection->count(); ++i)
@@ -1021,7 +1021,7 @@
        {
                m_view->deselectItems(false);
        }
-       currItem = m_canvas->itemUnderCursor(m->globalPos(), currItem, 
(m->modifiers() & SELECT_IN_GROUP));
+       currItem = m_canvas->itemUnderCursor(m->globalPosition(), currItem, 
(m->modifiers() & SELECT_IN_GROUP));
        if (currItem)
        {
                m_doc->m_Selection->delaySignalsOn();

Modified: trunk/Scribus/scribus/canvasmode_edittable.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_edittable.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_edittable.cpp      (original)
+++ trunk/Scribus/scribus/canvasmode_edittable.cpp      Sat Mar 12 16:00:07 2022
@@ -158,7 +158,7 @@
 {
        event->accept();
 
-       QPointF canvasPoint = 
m_canvas->globalToCanvas(event->globalPos()).toQPointF();
+       QPointF canvasPoint = 
m_canvas->globalToCanvas(event->globalPosition()).toQPointF();
        double threshold = m_doc->guidesPrefs().grabRadius / m_canvas->scale();
        TableHandle handle = m_table->hitTest(canvasPoint, threshold);
 
@@ -206,7 +206,8 @@
        PageItem_TextFrame* activeFrame;
 
        event->accept();
-       QPointF canvasPoint = 
m_canvas->globalToCanvas(event->globalPos()).toQPointF();
+       QPointF canvasPoint = 
m_canvas->globalToCanvas(event->globalPosition()).toQPointF();
+       QPoint globalPos = event->globalPosition().toPoint();
        double threshold = m_doc->guidesPrefs().grabRadius / m_canvas->scale();
        TableHandle handle = m_table->hitTest(canvasPoint, threshold);
        TableCell cell;
@@ -228,7 +229,7 @@
                                // Select row and pre-position text cursor
                                m_table->moveTo(cell);
                                m_table->selectRow(cell.row());
-                               m_view->slotSetCurs(event->globalPos().x(), 
event->globalPos().y());
+                               m_view->slotSetCurs(globalPos.x(), 
globalPos.y());
                                m_lastCursorPos = -1;
                                updateCanvas(true);
                                break;
@@ -250,7 +251,7 @@
                                // Select column and pre-position text cursor
                                m_table->moveTo(cell);
                                m_table->selectColumn(cell.column());
-                               m_view->slotSetCurs(event->globalPos().x(), 
event->globalPos().y());
+                               m_view->slotSetCurs(globalPos.x(), 
globalPos.y());
                                m_lastCursorPos = -1;
                                updateCanvas(true);
                                break;
@@ -268,7 +269,7 @@
                                // Move to the pressed cell and position the 
text cursor.
                                m_table->clearSelection();
                                m_table->moveTo(m_table->cellAt(canvasPoint));
-                               m_view->slotSetCurs(event->globalPos().x(), 
event->globalPos().y());
+                               m_view->slotSetCurs(globalPos.x(), 
globalPos.y());
                                m_lastCursorPos = 
m_table->activeCell().textFrame()->itemText.cursorPosition();
                                
m_view->m_ScMW->setTBvals(m_table->activeCell().textFrame());
                                makeLongTextCursorBlink();
@@ -292,8 +293,7 @@
        else if (event->button() == Qt::RightButton)
        {
                // Show the table popup menu.
-               //m_view->m_ScMW->scrMenuMgr->runMenuAtPos("ItemTable", 
event->globalPos());
-               const FPoint 
mousePointDoc(m_canvas->globalToCanvas(event->globalPos()));
+               const FPoint 
mousePointDoc(m_canvas->globalToCanvas(event->globalPosition()));
                createContextMenu(m_table, mousePointDoc.x(), 
mousePointDoc.y());
        }
 }
@@ -305,7 +305,7 @@
 
 void CanvasMode_EditTable::mouseDoubleClickEvent(QMouseEvent* event)
 {
-       TableCell hitCell = 
m_table->cellAt(m_canvas->globalToCanvas(event->globalPos()).toQPointF());
+       TableCell hitCell = 
m_table->cellAt(m_canvas->globalToCanvas(event->globalPosition()).toQPointF());
        if (!hitCell.isValid())
                return;
 
@@ -407,7 +407,7 @@
 
 void CanvasMode_EditTable::handleMouseDrag(QMouseEvent* event)
 {
-       TableCell hitCell = 
m_table->cellAt(m_canvas->globalToCanvas(event->globalPos()).toQPointF());
+       TableCell hitCell = 
m_table->cellAt(m_canvas->globalToCanvas(event->globalPosition()).toQPointF());
        TableCell activeCell = m_table->activeCell();
        PageItem_TextFrame* activeFrame = activeCell.textFrame();
 
@@ -416,7 +416,9 @@
                // Select text in active cell text frame.
                activeFrame->itemText.deselectAll();
                activeFrame->HasSel = false;
-               m_view->slotSetCurs(event->globalPos().x(), 
event->globalPos().y());
+
+               QPoint globalPos = event->globalPosition().toPoint();
+               m_view->slotSetCurs(globalPos.x(), globalPos.y());
 
                PageItem_TextFrame* newActiveFrame = 
m_table->activeCell().textFrame();
                if (activeFrame == newActiveFrame)

Modified: trunk/Scribus/scribus/canvasmode_framelinks.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_framelinks.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_framelinks.cpp     (original)
+++ trunk/Scribus/scribus/canvasmode_framelinks.cpp     Sat Mar 12 16:00:07 2022
@@ -157,7 +157,8 @@
        if ((m_canvas->m_viewMode.m_MouseButtonPressed) && (m->buttons() & 
Qt::LeftButton))
        {
                QPoint startP = m_canvas->canvasToGlobal(QPointF(m_Mxp, m_Myp));
-               
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(m->globalPos())).normalized());
+               QPoint globalPos = m->globalPosition().toPoint();
+               
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(globalPos)).normalized());
                m_view->setRedrawMarkerShown(true);
                m_view->HaveSelRect = true;
                return;

Modified: trunk/Scribus/scribus/canvasmode_measurements.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_measurements.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_measurements.cpp   (original)
+++ trunk/Scribus/scribus/canvasmode_measurements.cpp   Sat Mar 12 16:00:07 2022
@@ -88,7 +88,7 @@
        if (m_active)
        {
                m->accept();
-               adjustPoint(m->globalPos());            
+               adjustPoint(m->globalPosition());               
        }
        m_active = false;
        showValues();
@@ -99,12 +99,12 @@
        if (m_active)
        {
                m->accept();
-               adjustPoint(m->globalPos());
+               adjustPoint(m->globalPosition());
                showValues();
        }
        else
        {
-               FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+               FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
                m_canvas->displayCorrectedXYHUD(m->globalPosition(), 
mousePointDoc.x(), mousePointDoc.y());
        }
 }
@@ -113,24 +113,24 @@
 {
        m->accept();
        m_active = true;
-       adjustPoint(m->globalPos());
+       adjustPoint(m->globalPosition());
        m_start = m_current;
        m_startDoc = m_currentDoc;
        showValues();
 }
 
-void MeasurementsMode::adjustPoint(QPoint globalPoint)
+void MeasurementsMode::adjustPoint(QPointF globalPoint)
 {
-       QPoint point = globalPoint - (m_canvas->mapToParent(QPoint(0, 0)) + 
m_canvas->parentWidget()->mapToGlobal(QPoint(0, 0)));
-       m_canvas->update(QRect(m_start, 
m_current).normalized().adjusted(-1,-1,1,1));
+       QPointF point = globalPoint - (m_canvas->mapToParent(QPointF(0, 0)) + 
m_canvas->parentWidget()->mapToGlobal(QPointF(0, 0)));
+       m_canvas->update(QRect(m_start.toPoint(), 
m_current.toPoint()).normalized().adjusted(-1, -1, 1, 1));
        m_current = point;
-       m_currentDoc = m_canvas->localToCanvas(m_current) - 
FPoint(m_doc->currentPage()->xOffset(),m_doc->currentPage()->yOffset());
+       m_currentDoc = m_canvas->localToCanvas(m_current) - 
FPoint(m_doc->currentPage()->xOffset(), m_doc->currentPage()->yOffset());
 }
 
 void MeasurementsMode::showValues()
 {
        double dx = m_currentDoc.x() - m_startDoc.x();
        double dy = m_currentDoc.y() - m_startDoc.y();
-       m_palette->setValues(m_startDoc.x(), m_startDoc.y(), m_currentDoc.x(), 
m_currentDoc.y(), -xy2Deg(dx, dy), sqrt(dx*dx + dy*dy));
-       m_canvas->update(QRect(m_start, m_current).normalized());
+       m_palette->setValues(m_startDoc.x(), m_startDoc.y(), m_currentDoc.x(), 
m_currentDoc.y(), -xy2Deg(dx, dy), sqrt(dx * dx + dy * dy));
+       m_canvas->update(QRect(m_start.toPoint(), 
m_current.toPoint()).normalized());
 }

Modified: trunk/Scribus/scribus/canvasmode_measurements.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_measurements.h
==============================================================================
--- trunk/Scribus/scribus/canvasmode_measurements.h     (original)
+++ trunk/Scribus/scribus/canvasmode_measurements.h     Sat Mar 12 16:00:07 2022
@@ -54,11 +54,11 @@
        void setActive(bool);
 
 private:
-       void adjustPoint(QPoint point);
+       void adjustPoint(QPointF point);
        void showValues();
        bool m_active {false};
-       QPoint m_start;
-       QPoint m_current;
+       QPointF m_start;
+       QPointF m_current;
        FPoint m_startDoc;
        FPoint m_currentDoc;
        Measurements* m_palette {nullptr};

Modified: trunk/Scribus/scribus/canvasmode_nodeedit.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_nodeedit.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_nodeedit.cpp       (original)
+++ trunk/Scribus/scribus/canvasmode_nodeedit.cpp       Sat Mar 12 16:00:07 2022
@@ -264,11 +264,12 @@
 
 void CanvasMode_NodeEdit::mouseMoveEvent(QMouseEvent *m)
 {
-       PageItem *currItem;
-       FPoint npf;
        double sc = m_canvas->scale();
+       FPoint npf = m_canvas->globalToCanvas(m->globalPosition());
+
        m->accept();
-       npf = m_canvas->globalToCanvas(m->globalPos());
+
+       PageItem* currItem { nullptr };
        if (GetItem(&currItem))
        {
                if (m_doc->DragP)
@@ -293,14 +294,14 @@
        }
        else // shouldn't be in nodeedit mode now, should it??
        {
-               npf = m_canvas->globalToCanvas(m->globalPos());
+               npf = m_canvas->globalToCanvas(m->globalPosition());
                if ((m_canvas->m_viewMode.m_MouseButtonPressed) && 
(m->buttons() & Qt::LeftButton) && (m_GyM == -1) && (m_GxM == -1))
                {
                        if (!m_rectangleSelect)
                        {
                                m_rectangleSelect = new RectSelect(this);
                        }
-                       m_rectangleSelect->prepare(m->globalPos());
+                       m_rectangleSelect->prepare(m->globalPosition());
                        m_view->startGesture(m_rectangleSelect);
                        return;
                }
@@ -353,7 +354,7 @@
        m_doc->leaveDrag = false;
        m_MoveGX = m_MoveGY = false;
        m->accept();
-       m_view->registerMousePress(m->globalPos());
+       m_view->registerMousePress(m->globalPosition());
        QRect mpo;
        m_Mxp = m->position().x();
        m_Myp = m->position().y();
@@ -407,7 +408,7 @@
                if (m_doc->nodeEdit.hasNodeSelected() && 
(m_doc->nodeEdit.selectionCount() == 1))
                {
                        //FIXME:av
-                       FPoint newP = m_canvas->globalToCanvas(m->globalPos());
+                       FPoint newP = 
m_canvas->globalToCanvas(m->globalPosition());
                        currItem->OldB2 = currItem->width();
                        currItem->OldH2 = currItem->height();
                        FPointArray Clip;
@@ -507,7 +508,7 @@
        m_doc->nodeEdit.deselect();
 
        QTransform pm2 = currItem->getTransform();
-       npf2 = m_canvas->globalToCanvas(m->globalPos()).transformPoint(pm2, 
true);
+       npf2 = 
m_canvas->globalToCanvas(m->globalPosition()).transformPoint(pm2, true);
        if ((currItem->isSymbol() || currItem->isGroup()) && 
(!m_doc->nodeEdit.isContourLine()))
        {
                if (currItem->imageFlippedH())
@@ -526,7 +527,7 @@
                if (((m_doc->nodeEdit.edPoints()) && (i % 2 != 0)) || 
((!m_doc->nodeEdit.edPoints()) && (i % 2 == 0)))
                        continue;
                QPointF npf = pm2.map(Clip.pointQF(i));
-               if (m_canvas->hitsCanvasPoint(m->globalPos(), npf))
+               if (m_canvas->hitsCanvasPoint(m->globalPosition(), npf))
                {
                        m_doc->nodeEdit.setClre(i);
                        if ((m_doc->nodeEdit.edPoints()) && 
(m_doc->nodeEdit.selNode().contains(i) == 0))
@@ -952,7 +953,7 @@
                m_Dyp = qRound(m->y() / m_canvas->scale());  // + 
m_doc->minCanvasCoordinate.y());
                if (!m_rectangleSelect)
                        m_rectangleSelect = new RectSelect(this);
-               m_rectangleSelect->prepare(m->globalPos());
+               m_rectangleSelect->prepare(m->globalPosition());
                m_view->startGesture(m_rectangleSelect);
        }
        
@@ -993,7 +994,7 @@
                        if (((m_doc->nodeEdit.edPoints()) && (i % 2 != 0)) || 
((!m_doc->nodeEdit.edPoints()) && (i % 2 == 0)))
                                continue;
                        QPointF point = Clip.pointQF(i);
-                       if (m_canvas->hitsCanvasPoint(m->globalPos(), 
itemPos.map(point)))
+                       if (m_canvas->hitsCanvasPoint(m->globalPosition(), 
itemPos.map(point)))
                        {
                                if (m_doc->nodeEdit.submode() == 
NodeEditContext::MOVE_POINT)
                                        
m_view->setCursor(QCursor(Qt::SizeAllCursor));
@@ -1026,7 +1027,7 @@
                                for (double d = 0.0; d <= 1.0; d += delta)
                                {
                                        QPointF pl = Bez.pointAtPercent(d);
-                                       if 
(m_canvas->hitsCanvasPoint(m->globalPos(), FPoint(pl.x(), pl.y())))
+                                       if 
(m_canvas->hitsCanvasPoint(m->globalPosition(), FPoint(pl.x(), pl.y())))
                                        {
                                                if (m_doc->nodeEdit.submode() 
== NodeEditContext::MOVE_POINT)
                                                        
m_view->setCursor(IconManager::instance().loadCursor("handc.png"));
@@ -1054,7 +1055,7 @@
        {
                if (!m_rectangleSelect)
                        m_rectangleSelect = new RectSelect(this);
-               m_rectangleSelect->prepare(m->globalPos());
+               m_rectangleSelect->prepare(m->globalPosition());
                m_view->startGesture(m_rectangleSelect);
                return;
        }
@@ -1095,7 +1096,7 @@
                        int storedClRe = m_doc->nodeEdit.clre();
                        if (m_doc->nodeEdit.selectionCount() == 1)
                        {
-                               npf = m_canvas->globalToCanvas(m->globalPos());
+                               npf = 
m_canvas->globalToCanvas(m->globalPosition());
                                double nx = npf.x();
                                double ny = npf.y();
                                if (!m_doc->ApplyGuides(&nx, &ny) && 
!m_doc->ApplyGuides(&nx, &ny,true))
@@ -1199,7 +1200,7 @@
                }
                else
                {
-                       npf = m_canvas->globalToCanvas(m->globalPos());
+                       npf = m_canvas->globalToCanvas(m->globalPosition());
                        double nx = npf.x();
                        double ny = npf.y();
                        if (!m_doc->ApplyGuides(&nx, &ny) && 
!m_doc->ApplyGuides(&nx, &ny,true))

Modified: trunk/Scribus/scribus/canvasmode_normal.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_normal.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_normal.cpp (original)
+++ trunk/Scribus/scribus/canvasmode_normal.cpp Sat Mar 12 16:00:07 2022
@@ -138,8 +138,11 @@
 
 void CanvasMode_Normal::mouseDoubleClickEvent(QMouseEvent *m)
 {
+       const QPoint globalPos = m->globalPosition().toPoint();
        PageItem *currItem = nullptr;
+
        m->accept();
+
        m_canvas->m_viewMode.m_MouseButtonPressed = false;
        if (m_doc->drawAsPreview)
        {
@@ -225,7 +228,7 @@
                else if (currItem->itemType() == PageItem::TextFrame)
                {
                        // See if double click was on a frame handle
-                       FPoint p = m_canvas->globalToCanvas(m->globalPos());
+                       FPoint p = 
m_canvas->globalToCanvas(m->globalPosition());
                        Canvas::FrameHandle fh = 
m_canvas->frameHitTest(QPointF(p.x(),p.y()), currItem);
                        //CB if annotation, open the annotation dialog
                        if (currItem->isAnnotation())
@@ -239,7 +242,7 @@
                        else if (m_doc->appMode != modeEdit)
                        {
                                m_view->requestMode(modeEdit);
-                               m_view->slotSetCurs(m->globalPos().x(), 
m->globalPos().y());
+                               m_view->slotSetCurs(globalPos.x(), 
globalPos.y());
                                //used for updating Notes Styles Editor and 
menu actions for marks
                                //if cursor is in mark`s place
                                m_ScMW->setTBvals(currItem);
@@ -270,7 +273,7 @@
                else if (currItem->isTable())
                {
                        m_view->requestMode(modeEditTable);
-                       m_view->slotSetCurs(m->globalPos().x(), 
m->globalPos().y());
+                       m_view->slotSetCurs(globalPos.x(), globalPos.y());
                        m_ScMW->setTBvals(currItem);
                }
        }
@@ -280,16 +283,15 @@
 void CanvasMode_Normal::mouseMoveEvent(QMouseEvent *m)
 {
 //     qDebug()<<"CanvasMode_Normal::mouseMoveEvent";
-//     const double mouseX = m->globalX();
-//     const double mouseY = m->globalY();
-       const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       const QPoint globalPos = m->globalPosition().toPoint();
+       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
        
        bool wasLastPostOverGuide = m_lastPosWasOverGuide;
        m_lastPosWasOverGuide = false;
-       PageItem *currItem=nullptr;
+       PageItem *currItem = nullptr;
        bool erf = false;
        m->accept();
-//     qDebug() << "legacy mode move:" << m->x() << m->y() << 
m_canvas->globalToCanvas(m->globalPos()).x() << 
m_canvas->globalToCanvas(m->globalPos()).y();
+//     qDebug() << "legacy mode move:" << m->x() << m->y() << 
m_canvas->globalToCanvas(m->globalPosition()).x() << 
m_canvas->globalToCanvas(m->globalPosition()).y();
 //     emit MousePos(m->x()/m_canvas->scale(),// + 
m_doc->minCanvasCoordinate.x(), 
 //                               m->y()/m_canvas->scale()); // + 
m_doc->minCanvasCoordinate.y());
 
@@ -316,7 +318,7 @@
                }
                else
                {
-                       PageItem* hoveredItem = 
m_canvas->itemUnderCursor(m->globalPos(), nullptr, m->modifiers());
+                       PageItem* hoveredItem = 
m_canvas->itemUnderCursor(m->globalPosition(), nullptr, m->modifiers());
                        if (hoveredItem)
                                frameResizeHandle = 
m_canvas->frameHitTest(QPointF(mousePointDoc.x(), mousePointDoc.y()), 
hoveredItem);
                }
@@ -366,7 +368,7 @@
        if (!movingOrResizing && mouseIsOnPage)
        {
                PageItem* hoveredItem = nullptr;
-               hoveredItem = m_canvas->itemUnderCursor(m->globalPos(), 
hoveredItem, m->modifiers());
+               hoveredItem = m_canvas->itemUnderCursor(m->globalPosition(), 
hoveredItem, m->modifiers());
                if (hoveredItem)
                {
                        if (m_doc->drawAsPreview)
@@ -396,7 +398,7 @@
                                        if (toolT.isEmpty())
                                                toolT = 
hoveredItem->itemText.plainText();
                                        if (!toolT.isEmpty())
-                                               
QToolTip::showText(m->globalPos() + QPoint(5, 5), toolT, m_canvas);
+                                               QToolTip::showText(globalPos + 
QPoint(5, 5), toolT, m_canvas);
                                        if (hoveredItem != m_hoveredItem)
                                        {
                                                if (m_hoveredItem)
@@ -429,9 +431,9 @@
                        {
                                if (hoveredItem->isTextFrame() && 
hoveredItem->frameOverflows())
                                {
-                                       if 
(m_canvas->cursorOverTextFrameControl(m->globalPos(), hoveredItem))
+                                       if 
(m_canvas->cursorOverTextFrameControl(m->globalPosition(), hoveredItem))
                                        {
-                                               
QToolTip::showText(m->globalPos() + QPoint(5, 5), tr("Overflow Characters: %1 
(%2 White Spaces)\nClick to link to existing text frame or auto-create new 
linked text 
frame").arg(hoveredItem->frameOverflowCount()).arg(hoveredItem->frameOverflowBlankCount()),
 m_canvas);
+                                               QToolTip::showText(globalPos + 
QPoint(5, 5), tr("Overflow Characters: %1 (%2 White Spaces)\nClick to link to 
existing text frame or auto-create new linked text 
frame").arg(hoveredItem->frameOverflowCount()).arg(hoveredItem->frameOverflowBlankCount()),
 m_canvas);
                                        }
                                }
                                else
@@ -805,7 +807,7 @@
                        double newY = qRound(mousePointDoc.y());
                        m_mouseSavedPoint.setXY(newX, newY);
                        QPoint startP = 
m_canvas->canvasToGlobal(m_mousePressPoint);
-                       
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(m->globalPos())).normalized());
+                       
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(globalPos)).normalized());
                        m_view->setRedrawMarkerShown(true);
                        m_view->HaveSelRect = true;
                        return;
@@ -816,7 +818,8 @@
 void CanvasMode_Normal::mousePressEvent(QMouseEvent *m)
 {
 //     qDebug("CanvasMode_Normal::mousePressEvent");
-       const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       const QPoint globalPos = m->globalPosition().toPoint();
+       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
        PageItem *currItem;
 
        m_objectDeltaPos  = FPoint(0, 0);
@@ -828,7 +831,7 @@
        m_doc->DragP = false;
        m_doc->leaveDrag = false;
        m->accept();
-       m_view->registerMousePress(m->globalPos());
+       m_view->registerMousePress(m->globalPosition());
 
        if (m->button() == Qt::MiddleButton)
        {
@@ -909,7 +912,7 @@
                if (m_doc->m_Selection->isEmpty())
                {
                        m_mouseCurrentPoint = m_mousePressPoint = 
m_mouseSavedPoint = mousePointDoc;
-                       m_view->redrawMarker->setGeometry(m->globalPos().x(), 
m->globalPos().y(), 1, 1);
+                       m_view->redrawMarker->setGeometry(globalPos.x(), 
globalPos.y(), 1, 1);
                        m_view->setRedrawMarkerShown(true);
                }
                else
@@ -962,8 +965,8 @@
 #ifdef GESTURE_FRAME_PREVIEW
        clearPixmapCache();
 #endif // GESTURE_FRAME_PREVIEW
-       const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
-       PageItem *currItem;
+       const FPoint mousePointDoc = 
m_canvas->globalToCanvas(m->globalPosition());
+       PageItem *currItem { nullptr };
        m_canvas->m_viewMode.m_MouseButtonPressed = false;
        m_canvas->resetRenderMode();
        m->accept();
@@ -990,12 +993,12 @@
        // link text frames
        //<<#10116: Click on overflow icon to get into link frame mode
        PageItem* clickedItem = nullptr;
-       clickedItem = m_canvas->itemUnderCursor(m->globalPos(), clickedItem, 
m->modifiers());
+       clickedItem = m_canvas->itemUnderCursor(m->globalPosition(), 
clickedItem, m->modifiers());
        if (clickedItem && clickedItem->isTextFrame() && 
(!clickedItem->isAnnotation()) && (!m_doc->drawAsPreview))
        {
                if (clickedItem->frameOverflows())
                {
-                       if 
(m_canvas->cursorOverTextFrameControl(m->globalPos(), clickedItem))
+                       if 
(m_canvas->cursorOverTextFrameControl(m->globalPosition(), clickedItem))
                        {
                                m_view->requestMode(modeLinkFrames);
                                return;
@@ -1261,7 +1264,7 @@
                                {
                                        if (currItem->annotation().IsOpen())
                                        {
-                                               if 
(m_canvas->cursorOverFrameControl(m->globalPos(), QRectF(245, 20, 11, 11), 
currItem))
+                                               if 
(m_canvas->cursorOverFrameControl(m->globalPosition(), QRectF(245, 20, 11, 11), 
currItem))
                                                {
                                                        
currItem->annotation().setOpen(false);
                                                        
currItem->asTextFrame()->setTextAnnotationOpen(false);
@@ -1557,7 +1560,7 @@
        QTransform p;
        PageItem *currItem;
        m_canvas->m_viewMode.m_MouseButtonPressed = true;
-       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
+       FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPosition());
        m_mouseCurrentPoint  = mousePointDoc;
 //     double grabRadius = m_doc->guidesPrefs().grabRadius / m_canvas->scale();
        int MxpS = static_cast<int>(mousePointDoc.x()); 
//m->x()/m_canvas->scale() + 0*m_doc->minCanvasCoordinate.x());
@@ -1581,7 +1584,7 @@
                }
                else
                {
-                       PageItem* hoveredItem = 
m_canvas->itemUnderCursor(m->globalPos(), nullptr, m->modifiers());
+                       PageItem* hoveredItem = 
m_canvas->itemUnderCursor(m->globalPosition(), nullptr, m->modifiers());
                        if (hoveredItem)
                                frameResizeHandle = 
m_canvas->frameHitTest(QPointF(mousePointDoc.x(), mousePointDoc.y()), 
hoveredItem);
                }
@@ -1678,7 +1681,7 @@
                m_view->deselectItems(false);
        }
 
-       currItem = m_canvas->itemUnderCursor(m->globalPos(), currItem, 
((m->modifiers() & SELECT_IN_GROUP) || (m_doc->drawAsPreview && 
!m_doc->editOnPreview)));
+       currItem = m_canvas->itemUnderCursor(m->globalPosition(), currItem, 
((m->modifiers() & SELECT_IN_GROUP) || (m_doc->drawAsPreview && 
!m_doc->editOnPreview)));
        if (currItem)
        {
                m_doc->m_Selection->delaySignalsOn();
@@ -1750,7 +1753,7 @@
                }
                else
                {
-                       PageItem* hoveredItem = 
m_canvas->itemUnderCursor(m->globalPos(), nullptr, m->modifiers());
+                       PageItem* hoveredItem = 
m_canvas->itemUnderCursor(m->globalPosition(), nullptr, m->modifiers());
                        if (hoveredItem)
                                frameResizeHandle = 
m_canvas->frameHitTest(QPointF(mousePointDoc.x(), mousePointDoc.y()), 
hoveredItem);
                }

Modified: trunk/Scribus/scribus/canvasmode_rotate.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/canvasmode_rotate.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_rotate.cpp (original)
+++ trunk/Scribus/scribus/canvasmode_rotate.cpp Sat Mar 12 16:00:07 2022
@@ -431,7 +431,8 @@
                if ((m_canvas->m_viewMode.m_MouseButtonPressed) && 
(m->buttons() & Qt::LeftButton))
                {
                        QPoint startP = 
m_canvas->canvasToGlobal(m_canvasPressCoord);
-                       
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(m->globalPos())).normalized());
+                       QPoint globalPos = m->globalPosition().toPoint();
+                       
m_view->redrawMarker->setGeometry(QRect(m_view->mapFromGlobal(startP), 
m_view->mapFromGlobal(globalPos)).normalized());
                        m_view->setRedrawMarkerShown(true);
                        m_view->HaveSelRect = true;
                }

Modified: trunk/Scribus/scribus/chartableview.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/chartableview.cpp
==============================================================================
--- trunk/Scribus/scribus/chartableview.cpp     (original)
+++ trunk/Scribus/scribus/chartableview.cpp     Sat Mar 12 16:00:07 2022
@@ -94,7 +94,7 @@
                else
                {
                        deleteAct->setData(index);
-                       actionMenu->popup(event->globalPos());
+                       actionMenu->popup(event->globalPosition().toPoint());
                }
        }
 }

Modified: trunk/Scribus/scribus/ui/hruler.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/ui/hruler.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/hruler.cpp (original)
+++ trunk/Scribus/scribus/ui/hruler.cpp Sat Mar 12 16:00:07 2022
@@ -202,7 +202,7 @@
                {
                        qApp->setOverrideCursor(QCursor(Qt::SplitVCursor));
                        m_view->startGesture(rulerGesture);
-                       m_view->registerMousePress(m->globalPos());
+                       m_view->registerMousePress(m->globalPosition());
                }
        }
 }

Modified: trunk/Scribus/scribus/ui/rulermover.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/ui/rulermover.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/rulermover.cpp     (original)
+++ trunk/Scribus/scribus/ui/rulermover.cpp     Sat Mar 12 16:00:07 2022
@@ -41,7 +41,7 @@
 #include "ui/guidemanager.h"
 
 
-RulerMover::RulerMover(ScribusView *pa) : QWidget(pa)
+RulerMover::RulerMover(ScribusView *pa) : QWidget(pa), m_currView(pa)
 {
        setBackgroundRole(QPalette::Window);
        setAutoFillBackground(true);
@@ -49,45 +49,43 @@
        palette.setBrush(QPalette::Window, QColor(240, 240, 240));
        palette.setBrush(backgroundRole(), 
QBrush(IconManager::instance().loadPixmap("mover.png")));
        setPalette(palette);
-       currView = pa;
-       rulerGesture = new RulerGesture(currView, RulerGesture::ORIGIN);
-       Mpressed = false;
+       m_rulerGesture = new RulerGesture(m_currView, RulerGesture::ORIGIN);
 }
 
 void RulerMover::mouseDoubleClickEvent(QMouseEvent *)
 {
-       currView->m_doc->rulerXoffset = 0;
-       currView->m_doc->rulerYoffset = 0;
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 0;
+       m_currView->m_doc->rulerYoffset = 0;
+       m_currView->DrawNew();
 }
 
 void RulerMover::mousePressEvent(QMouseEvent *m)
 {
        if (m->button() == Qt::LeftButton)
        {
-               Mpressed = true;
-               qApp->setOverrideCursor(QCursor(Qt::CrossCursor));
-               currView->startGesture(rulerGesture);
-               currView->registerMousePress(m->globalPos());
+               m_mousePressed = true;
+               QApplication::setOverrideCursor(QCursor(Qt::CrossCursor));
+               m_currView->startGesture(m_rulerGesture);
+               m_currView->registerMousePress(m->globalPosition());
        }
        QWidget::mousePressEvent(m);
 }
 
 void RulerMover::mouseReleaseEvent(QMouseEvent *m)
 {
-       qApp->restoreOverrideCursor();
-       if (Mpressed)
+       QApplication::restoreOverrideCursor();
+       if (m_mousePressed)
        {
-               rulerGesture->mouseReleaseEvent(m);
-               Mpressed = false;
+               m_rulerGesture->mouseReleaseEvent(m);
+               m_mousePressed = false;
        }
        else if (m->button() == Qt::RightButton)
        {
                QMenu *pmen = new QMenu();
                QMenu *pmen2 = nullptr;
                pmen->addAction( tr("Reset Rulers"), this, SLOT(resetRulers()));
-               
pmen->addAction(currView->m_doc->scMW()->scrActions["viewRulerMode"]);
-               if (currView->m_doc->guidesPrefs().rulerMode)
+               
pmen->addAction(m_currView->m_doc->scMW()->scrActions["viewRulerMode"]);
+               if (m_currView->m_doc->guidesPrefs().rulerMode)
                {
                        pmen2 = new QMenu();
                        if (pmen2)
@@ -107,64 +105,64 @@
                delete pmen;
                delete pmen2;
        }
-       if (currView->m_doc->guidesPrefs().guidesShown)
-               currView->m_doc->scMW()->guidePalette->setupPage();
+       if (m_currView->m_doc->guidesPrefs().guidesShown)
+               m_currView->m_doc->scMW()->guidePalette->setupPage();
        QWidget::mouseReleaseEvent(m);
 }
 
 void RulerMover::mouseMoveEvent(QMouseEvent *m)
 {
-       if (Mpressed)
-               rulerGesture->mouseMoveEvent(m);
+       if (m_mousePressed)
+               m_rulerGesture->mouseMoveEvent(m);
        QWidget::mouseMoveEvent(m);
 }
 
 void RulerMover::resetRulers()
 {
-       currView->m_doc->rulerXoffset = 0;
-       currView->m_doc->rulerYoffset = 0;
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 0;
+       m_currView->m_doc->rulerYoffset = 0;
+       m_currView->DrawNew();
 }
 
 void RulerMover::moveRulerTopRight()
 {
-       currView->m_doc->rulerXoffset = currView->m_doc->currentPage()->width();
-       currView->m_doc->rulerYoffset = 0;
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 
m_currView->m_doc->currentPage()->width();
+       m_currView->m_doc->rulerYoffset = 0;
+       m_currView->DrawNew();
 }
 
 void RulerMover::moveRulerBottomLeft()
 {
-       currView->m_doc->rulerXoffset = 0;
-       currView->m_doc->rulerYoffset = 
currView->m_doc->currentPage()->height();
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 0;
+       m_currView->m_doc->rulerYoffset = 
m_currView->m_doc->currentPage()->height();
+       m_currView->DrawNew();
 }
 
 void RulerMover::moveRulerBottomRight()
 {
-       currView->m_doc->rulerXoffset = currView->m_doc->currentPage()->width();
-       currView->m_doc->rulerYoffset = 
currView->m_doc->currentPage()->height();
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 
m_currView->m_doc->currentPage()->width();
+       m_currView->m_doc->rulerYoffset = 
m_currView->m_doc->currentPage()->height();
+       m_currView->DrawNew();
 }
 
 void RulerMover::moveRulerCenter()
 {
-       currView->m_doc->rulerXoffset = currView->m_doc->currentPage()->width() 
/ 2.0;
-       currView->m_doc->rulerYoffset = 
currView->m_doc->currentPage()->height() / 2.0;
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 
m_currView->m_doc->currentPage()->width() / 2.0;
+       m_currView->m_doc->rulerYoffset = 
m_currView->m_doc->currentPage()->height() / 2.0;
+       m_currView->DrawNew();
 }
 
 void RulerMover::moveRulerTopCenter()
 {
-       currView->m_doc->rulerXoffset = currView->m_doc->currentPage()->width() 
/ 2.0;
-       currView->m_doc->rulerYoffset = 0;
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 
m_currView->m_doc->currentPage()->width() / 2.0;
+       m_currView->m_doc->rulerYoffset = 0;
+       m_currView->DrawNew();
 }
 
 void RulerMover::moveRulerBottomCenter()
 {
-       currView->m_doc->rulerXoffset = currView->m_doc->currentPage()->width() 
/ 2.0;
-       currView->m_doc->rulerYoffset = 
currView->m_doc->currentPage()->height();
-       currView->DrawNew();
+       m_currView->m_doc->rulerXoffset = 
m_currView->m_doc->currentPage()->width() / 2.0;
+       m_currView->m_doc->rulerYoffset = 
m_currView->m_doc->currentPage()->height();
+       m_currView->DrawNew();
 }
 

Modified: trunk/Scribus/scribus/ui/rulermover.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/ui/rulermover.h
==============================================================================
--- trunk/Scribus/scribus/ui/rulermover.h       (original)
+++ trunk/Scribus/scribus/ui/rulermover.h       Sat Mar 12 16:00:07 2022
@@ -38,11 +38,12 @@
 public:
        RulerMover(ScribusView *pa);
        ~RulerMover() {};
-       void mouseDoubleClickEvent(QMouseEvent *);
-       void mousePressEvent(QMouseEvent *m);
-       void mouseReleaseEvent(QMouseEvent *m);
-       void mouseMoveEvent(QMouseEvent *m);
-       bool Mpressed;
+
+protected:
+       void mouseDoubleClickEvent(QMouseEvent*) override;
+       void mousePressEvent(QMouseEvent* m) override;
+       void mouseReleaseEvent(QMouseEvent* m) override;
+       void mouseMoveEvent(QMouseEvent* m) override;
 
 private slots:
        void resetRulers();
@@ -54,8 +55,9 @@
        void moveRulerBottomCenter();
 
 private: // Private attributes
-       ScribusView *currView;
-       RulerGesture* rulerGesture;
+       bool m_mousePressed { false };
+       ScribusView *m_currView { nullptr };
+       RulerGesture* m_rulerGesture { nullptr };
 };
 
 #endif

Modified: trunk/Scribus/scribus/ui/storyeditor.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25019&path=/trunk/Scribus/scribus/ui/storyeditor.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/storyeditor.cpp    (original)
+++ trunk/Scribus/scribus/ui/storyeditor.cpp    Sat Mar 12 16:00:07 2022
@@ -139,7 +139,7 @@
 
 void SideBar::mouseReleaseEvent(QMouseEvent *m)
 {
-       QPoint globalPos = m->globalPos();
+       QPoint globalPos = m->globalPosition().toPoint();
        QPoint viewPos   = m_editor->viewport()->mapFromGlobal(globalPos);
        int p = m_editor->cursorForPosition(QPoint(2, viewPos.y())).position();
        currentPar = m_editor->StyledText.nrOfParagraph(p);


_______________________________________________
scribus-commit mailing list
[email protected]
http://lists.scribus.net/mailman/listinfo/scribus-commit

Reply via email to