Author: jghali
Date: Sat Mar 12 21:39:55 2022
New Revision: 25025

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=25025
Log:
Replace quite a few repaint() calls by more adapted update() calls

Modified:
    trunk/Scribus/scribus/ui/customfdialog.cpp
    trunk/Scribus/scribus/ui/dasheditor.cpp
    trunk/Scribus/scribus/ui/picsearch.cpp
    trunk/Scribus/scribus/ui/propertiespalette.cpp
    trunk/Scribus/scribus/ui/propertiespalette_image.cpp
    trunk/Scribus/scribus/ui/propertiespalette_line.cpp
    trunk/Scribus/scribus/ui/propertiespalette_text.cpp
    trunk/Scribus/scribus/ui/propertiespalette_xyz.cpp
    trunk/Scribus/scribus/ui/propertywidget_fontfeatures.cpp
    trunk/Scribus/scribus/ui/propertywidget_hyphenation.cpp
    trunk/Scribus/scribus/ui/smtabruler.cpp
    trunk/Scribus/scribus/ui/tabruler.cpp
    trunk/Scribus/scribus/ui/textpalette.cpp

Modified: trunk/Scribus/scribus/ui/customfdialog.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/customfdialog.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/customfdialog.cpp  (original)
+++ trunk/Scribus/scribus/ui/customfdialog.cpp  Sat Mar 12 21:39:55 2022
@@ -193,7 +193,7 @@
                p.drawText(2, h - 5, tr("Colorspace:") + " " + cSpace);
                p.end();
                setPixmap(pm);
-               repaint();
+               update();
        }
        else if (allFormatsV.contains(ext.toUtf8()))
        {
@@ -223,7 +223,7 @@
                p.drawText(2, h - 5, desc);
                p.end();
                setPixmap(pm);
-               repaint();
+               update();
        }
        else if (ext.toUtf8() == "sce")
        {
@@ -249,7 +249,7 @@
                p.drawText(2, h - 5, desc);
                p.end();
                setPixmap(pm);
-               repaint();
+               update();
                delete pre;
        }
        else
@@ -570,9 +570,9 @@
        filePreview->setVisible(!m_previewIsShown);
        QCoreApplication::processEvents();
        filePreview->setVisible(m_previewIsShown);
-       fileDialog->repaint();
+       fileDialog->update();
        QCoreApplication::processEvents();
-       repaint();
+       update();
 }
 
 void CustomFDialog::setSelection(const QString& fileName)

Modified: trunk/Scribus/scribus/ui/dasheditor.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/dasheditor.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/dasheditor.cpp     (original)
+++ trunk/Scribus/scribus/ui/dasheditor.cpp     Sat Mar 12 21:39:55 2022
@@ -129,7 +129,7 @@
                {
                        m_currentStop = i;
                        emit currStep(m_dashValues[m_currentStop]);
-                       repaint();
+                       update();
                        return;
                }
        }
@@ -154,7 +154,7 @@
                m_currentStop = 0;
                m_onlySelect = true;
                m_mousePressed = false;
-               repaint();
+               update();
                emit currStep(m_dashValues[m_currentStop]);
                emit dashChanged();
                return;
@@ -195,7 +195,7 @@
                        startX += w;
                }
                m_onlySelect = true;
-               repaint();
+               update();
                emit currStep(m_dashValues[m_currentStop]);
                emit dashChanged();
        }
@@ -244,7 +244,7 @@
                        startX += w;
                }
                m_onlySelect = true;
-               repaint();
+               update();
                startX = 0.0;
                for (int i = 0; i < m_currentStop; ++i)
                {
@@ -279,7 +279,7 @@
                return;
        m_dashValues[m_currentStop] = t;
        m_onlySelect = true;
-       repaint();
+       update();
        emit dashChanged();
 }
 

Modified: trunk/Scribus/scribus/ui/picsearch.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/picsearch.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/picsearch.cpp      (original)
+++ trunk/Scribus/scribus/ui/picsearch.cpp      Sat Mar 12 21:39:55 2022
@@ -125,7 +125,7 @@
                        cSpace=colorSpaceText(im.imgInfo.colorspace);
                p.drawText(2, h-5, tr("Colorspace:")+" "+cSpace);
                p.end();
-               repaint();
+               update();
        }
        previewLabel->setPixmap(pm);
 }

Modified: trunk/Scribus/scribus/ui/propertiespalette.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertiespalette.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertiespalette.cpp      (original)
+++ trunk/Scribus/scribus/ui/propertiespalette.cpp      Sat Mar 12 21:39:55 2022
@@ -511,7 +511,7 @@
        xyzPal->setLineMode(mode);
        xyzPal->showWH(m_item->width(), m_item->height());
        updateGeometry();
-       repaint();
+       update();
 }
 
 void PropertiesPalette::handleNewShape(int frameType)

Modified: trunk/Scribus/scribus/ui/propertiespalette_image.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertiespalette_image.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertiespalette_image.cpp        (original)
+++ trunk/Scribus/scribus/ui/propertiespalette_image.cpp        Sat Mar 12 
21:39:55 2022
@@ -420,7 +420,6 @@
                setCurrentItem(currItem);
        }
        updateGeometry();
-       //repaint();
 }
 
 void PropertiesPalette_Image::handleUpdateRequest(int updateFlags)

Modified: trunk/Scribus/scribus/ui/propertiespalette_line.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertiespalette_line.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertiespalette_line.cpp (original)
+++ trunk/Scribus/scribus/ui/propertiespalette_line.cpp Sat Mar 12 21:39:55 2022
@@ -224,7 +224,6 @@
                setCurrentItem(currItem);
        }
        updateGeometry();
-       //repaint();
 }
 
 void PropertiesPalette_Line::handleUpdateRequest(int updateFlags)

Modified: trunk/Scribus/scribus/ui/propertiespalette_text.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertiespalette_text.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertiespalette_text.cpp (original)
+++ trunk/Scribus/scribus/ui/propertiespalette_text.cpp Sat Mar 12 21:39:55 2022
@@ -275,7 +275,6 @@
                setCurrentItem(currItem);
        }
        updateGeometry();
-       //repaint();
 }
 
 void PropertiesPalette_Text::handleUpdateRequest(int updateFlags)

Modified: trunk/Scribus/scribus/ui/propertiespalette_xyz.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertiespalette_xyz.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertiespalette_xyz.cpp  (original)
+++ trunk/Scribus/scribus/ui/propertiespalette_xyz.cpp  Sat Mar 12 21:39:55 2022
@@ -511,7 +511,6 @@
                setCurrentItem(currItem);
        }
        updateGeometry();
-       //repaint();
 }
 
 void PropertiesPalette_XYZ::unitChange()

Modified: trunk/Scribus/scribus/ui/propertywidget_fontfeatures.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertywidget_fontfeatures.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertywidget_fontfeatures.cpp    (original)
+++ trunk/Scribus/scribus/ui/propertywidget_fontfeatures.cpp    Sat Mar 12 
21:39:55 2022
@@ -279,7 +279,7 @@
        PageItem* currItem = currentItemFromSelection();
        setCurrentItem(currItem);
        updateGeometry();
-       repaint();
+       update();
 }
 
 void PropertyWidget_FontFeatures::updateCharStyle(const CharStyle& charStyle)

Modified: trunk/Scribus/scribus/ui/propertywidget_hyphenation.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/propertywidget_hyphenation.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertywidget_hyphenation.cpp     (original)
+++ trunk/Scribus/scribus/ui/propertywidget_hyphenation.cpp     Sat Mar 12 
21:39:55 2022
@@ -114,7 +114,7 @@
        PageItem* currItem = currentItemFromSelection();
        setCurrentItem(currItem);
        updateGeometry();
-       repaint();
+       update();
 }
 
 void PropertyWidget_Hyphenation::updateCharStyle(const CharStyle& charStyle)

Modified: trunk/Scribus/scribus/ui/smtabruler.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/smtabruler.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/smtabruler.cpp     (original)
+++ trunk/Scribus/scribus/ui/smtabruler.cpp     Sat Mar 12 21:39:55 2022
@@ -57,33 +57,28 @@
 
 void SMTabruler::unitChange(int unitIndex)
 {
-       this->blockSignals(true);
+       QSignalBlocker thisBlocker(this);
+       QSignalBlocker firstLineSpinBlocker(firstLineSpin);
+       QSignalBlocker leftIndentSpinBlocker(leftIndentSpin);
+       QSignalBlocker rightIndentSpinBlocker(rightIndentSpin);
+       QSignalBlocker tabDataBlocker(tabData);
+
        Tabruler::setTabs(ruler->tabValues, unitIndex);
-       Tabruler::repaint();
-
-       firstLineSpin->blockSignals(true);
-       leftIndentSpin->blockSignals(true);
-       rightIndentSpin->blockSignals(true);
-       tabData->blockSignals(true);
+       Tabruler::update();
 
        firstLineSpin->setNewUnit(unitIndex);
        leftIndentSpin->setNewUnit(unitIndex);
        rightIndentSpin->setNewUnit(unitIndex);
        tabData->setNewUnit(unitIndex);
 
-       firstLineSpin->blockSignals(false);
-       leftIndentSpin->blockSignals(false);
-       rightIndentSpin->blockSignals(false);
-       tabData->blockSignals(false);
-
        m_unitIndex = unitIndex;
-       this->blockSignals(false);
 }
 
 void SMTabruler::setTabs(const QList<ParagraphStyle::TabRecord>& Tabs, int 
unitIndex)
 {
        disconnect(this, SIGNAL(tabsChanged()), this, SLOT(slotTabsChanged()));
        disconnect(this, SIGNAL(mouseReleased()), this, 
SLOT(slotTabsChanged()));
+
        m_hasParent = false;
        m_parentButton->hide();
        Tabruler::setTabs(Tabs, unitIndex);
@@ -98,6 +93,7 @@
 {
        disconnect(this, SIGNAL(tabsChanged()), this, SLOT(slotTabsChanged()));
        disconnect(this, SIGNAL(mouseReleased()), this, 
SLOT(slotTabsChanged()));
+
        m_hasParent  = true;
        m_unitIndex = unitIndex;
        if (isParentValue)
@@ -105,7 +101,7 @@
        else
                m_parentButton->show();
        Tabruler::setTabs(Tabs, unitIndex);
-       Tabruler::repaint();
+       Tabruler::update();
        firstLineSpin->setNewUnit(unitIndex);
        leftIndentSpin->setNewUnit(unitIndex);
        rightIndentSpin->setNewUnit(unitIndex);
@@ -241,7 +237,7 @@
        if (ret && m_hasParent)
        {
                setTabs(m_parentTabs, m_unitIndex, true);
-               Tabruler::repaint();
+               Tabruler::update();
                m_parentButton->hide();
                m_useParentTabs = false;
        }

Modified: trunk/Scribus/scribus/ui/tabruler.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/tabruler.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/tabruler.cpp       (original)
+++ trunk/Scribus/scribus/ui/tabruler.cpp       Sat Mar 12 21:39:55 2022
@@ -67,7 +67,7 @@
        m_iter2 = unitRulerGetIter2FromIndex(unitIndex);
        tabValues = tabs;
        actTab    = -1;
-       repaint();
+       update();
 }
 
 void RulerT::paintEvent(QPaintEvent *)
@@ -207,7 +207,7 @@
                                emit typeChanged(tabValues[actTab].tabType);
                                emit tabMoved(tabValues[actTab].tabPosition);
                                emit 
fillCharChanged(tabValues[actTab].tabFillChar);
-                               repaint();
+                               update();
                                return;
                        }
                }
@@ -220,7 +220,7 @@
                actTab = 0;
                rulerCode = 3;
                updateTabList();
-               repaint();
+               update();
                emit newTab();
                emit typeChanged(tabValues[actTab].tabType);
                emit tabMoved(tabValues[actTab].tabPosition);
@@ -245,7 +245,7 @@
                                if (tabValues[actTab].tabType > 4)
                                        tabValues[actTab].tabType = 0;
                                emit typeChanged(tabValues[actTab].tabType);
-                               repaint();
+                               update();
                        }
                }
        }
@@ -263,7 +263,7 @@
                        }
                        else
                                emit noTabs();
-                       repaint();
+                       update();
                }
        }
        rulerCode = 0;
@@ -288,7 +288,7 @@
                                if (firstLine + leftIndent > m_rulerWidth)
                                        firstLine  = m_rulerWidth - leftIndent;
                                emit firstLineMoved(firstLine);
-                               repaint();
+                               update();
                                break;
                        case 2:
                                oldInd = leftIndent+firstLine;
@@ -300,7 +300,7 @@
                                firstLine = oldInd - leftIndent;
                                emit leftIndentMoved(leftIndent);
                                emit firstLineMoved(firstLine);
-                               repaint();
+                               update();
                                break;
                        case 3:
                                tabValues[actTab].tabPosition -= mouseX - mPosX;
@@ -310,7 +310,7 @@
                                        tabValues[actTab].tabPosition = 
m_rulerWidth - 1;
                                updateTabList();
                                emit tabMoved(tabValues[actTab].tabPosition);
-                               repaint();
+                               update();
                                break;
                        default:
                                break;
@@ -403,7 +403,7 @@
                offsetIncrement = 30;
        if (offset + width() > static_cast<int>(m_rulerWidth))
                offset -= 5;
-       repaint();
+       update();
 }
 
 void RulerT::decreaseOffset()
@@ -414,7 +414,7 @@
                offsetIncrement = 30;
        if (offset < 0)
                offset = 0;
-       repaint();
+       update();
 }
 
 void RulerT::changeTab(int t)
@@ -422,7 +422,7 @@
        if (actTab < 0 || actTab >= tabValues.count())
                return;
        tabValues[actTab].tabType = t;
-       repaint();
+       update();
 }
 
 void RulerT::changeTabChar(QChar t)
@@ -430,7 +430,7 @@
        if (actTab < 0 || actTab >= tabValues.count())
                return;
        tabValues[actTab].tabFillChar = t;
-       repaint();
+       update();
 }
 
 void RulerT::moveTab(double t)
@@ -439,7 +439,7 @@
                return;
        tabValues[actTab].tabPosition = t;
        updateTabList();
-       repaint();
+       update();
 }
 
 void RulerT::removeActTab()
@@ -457,7 +457,7 @@
                else
                        emit noTabs();
        }
-       repaint();
+       update();
 }
 
 void RulerT::moveFirstLine(double t)
@@ -473,7 +473,7 @@
                firstLine = m_rulerWidth - leftIndent;
                emit firstLineMoved(firstLine);
        }
-       repaint();
+       update();
 }
 
 void RulerT::moveLeftIndent(double t)
@@ -487,7 +487,7 @@
        }
        firstLine = oldInd - leftIndent;
        emit firstLineMoved(firstLine);
-       repaint();
+       update();
 }
 
 Tabruler::Tabruler( QWidget* parent, bool haveFirst, int unit, const 
QList<ParagraphStyle::TabRecord>& tabs, double wid ): QWidget( parent )
@@ -774,7 +774,7 @@
 void Tabruler::clearAll()
 {
        ruler->tabValues.clear();
-       ruler->repaint();
+       ruler->update();
        lastTabRemoved();
        emit tabrulerChanged();
        emit tabsChanged();

Modified: trunk/Scribus/scribus/ui/textpalette.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=25025&path=/trunk/Scribus/scribus/ui/textpalette.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/textpalette.cpp    (original)
+++ trunk/Scribus/scribus/ui/textpalette.cpp    Sat Mar 12 21:39:55 2022
@@ -310,7 +310,7 @@
        if (!m_ScMW || m_ScMW->scriptIsRunning())
                return;
        updateGeometry();
-       repaint();
+       update();
 }
 
 void TextPalette::updateColorList()


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

Reply via email to