Author: craig
Date: Mon Dec 12 22:14:13 2016
New Revision: 21646

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=21646
Log:
Some clean up work

Modified:
    trunk/Scribus/scribus/canvasmode_normal.cpp
    trunk/Scribus/scribus/chartablemodel.h
    trunk/Scribus/scribus/ui/charselect.cpp
    trunk/Scribus/scribus/ui/charselect.h
    trunk/Scribus/scribus/ui/charselect.ui
    trunk/Scribus/scribus/ui/charselectenhanced.cpp
    trunk/Scribus/scribus/ui/propertywidget_distance.cpp

Modified: trunk/Scribus/scribus/canvasmode_normal.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/canvasmode_normal.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_normal.cpp (original)
+++ trunk/Scribus/scribus/canvasmode_normal.cpp Mon Dec 12 22:14:13 2016
@@ -821,6 +821,7 @@
 
 void CanvasMode_Normal::mousePressEvent(QMouseEvent *m)
 {
+//     qDebug("CanvasMode_Normal::mousePressEvent");
        const FPoint mousePointDoc = m_canvas->globalToCanvas(m->globalPos());
        PageItem *currItem;
 
@@ -994,7 +995,7 @@
 
 void CanvasMode_Normal::mouseReleaseEvent(QMouseEvent *m)
 {
-//     qDebug("CanvasMode_Normal::mouseReleaseEvent");
+//     qDebug("CanvasMode_Normal::mouseReleaseEvent");
 #ifdef GESTURE_FRAME_PREVIEW
         clearPixmapCache();
 #endif // GESTURE_FRAME_PREVIEW
@@ -1006,15 +1007,18 @@
        m_view->setRedrawMarkerShown(false);
 //     m_view->stopDragTimer();
        //m_canvas->update(); //ugly in a mouseReleaseEvent!!!!!!!
-       if ((!GetItem(&currItem)) && (m->button() == Qt::RightButton) && 
(!m_doc->DragP) && (!m_doc->drawAsPreview))
-       {
-               createContextMenu(NULL, mousePointDoc.x(), mousePointDoc.y());
-               return;
-       }
-       if ((GetItem(&currItem)) && (m->button() == Qt::RightButton) && 
(!m_doc->DragP) && (!(m_doc->drawAsPreview && !m_doc->editOnPreview)))
-       {
-               createContextMenu(currItem, mousePointDoc.x(), 
mousePointDoc.y());
-               return;
+       if (m->button() == Qt::RightButton && (!m_doc->DragP) )
+       {
+               if ((!GetItem(&currItem)) && (!m_doc->drawAsPreview))
+               {
+                       createContextMenu(NULL, mousePointDoc.x(), 
mousePointDoc.y());
+                       return;
+               }
+               if ((GetItem(&currItem)) && (!(m_doc->drawAsPreview && 
!m_doc->editOnPreview)))
+               {
+                       createContextMenu(currItem, mousePointDoc.x(), 
mousePointDoc.y());
+                       return;
+               }
        }
        //<<#10116: Click on overflow icon to get into link frame mode
        PageItem* clickedItem=NULL;

Modified: trunk/Scribus/scribus/chartablemodel.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/chartablemodel.h
==============================================================================
--- trunk/Scribus/scribus/chartablemodel.h      (original)
+++ trunk/Scribus/scribus/chartablemodel.h      Mon Dec 12 22:14:13 2016
@@ -50,7 +50,7 @@
        void addCharacter(QString ch);
        CharClassDef characters() {
                return m_characters;
-       };
+       }
 
        QStringList fonts() { return m_fonts; }
 

Modified: trunk/Scribus/scribus/ui/charselect.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/ui/charselect.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/charselect.cpp     (original)
+++ trunk/Scribus/scribus/ui/charselect.cpp     Mon Dec 12 22:14:13 2016
@@ -30,7 +30,7 @@
 
        paletteFileMask = tr("Scribus Char Palette (*.ucp);;All Files (*)");
 
-       
hideButton->setIcon(IconManager::instance()->loadIcon("16/insert-table.png"));
+       
enhancedDialogButton->setIcon(IconManager::instance()->loadIcon("16/insert-table.png"));
        unicodeButton->setIcon(IconManager::instance()->loadIcon("find.png"));
        
uniLoadButton->setIcon(IconManager::instance()->loadIcon("16/document-open.png"));
        
uniSaveButton->setIcon(IconManager::instance()->loadIcon("16/document-save.png"));
@@ -49,7 +49,7 @@
        connect(m_userTable->selectionModel(), SIGNAL(selectionChanged(const 
QItemSelection& ,const QItemSelection&)), this, 
SLOT(slot_selectionChanged(const QItemSelection&, const QItemSelection&)) );
        connect(m_userTableModel, SIGNAL(rowAppended()), m_userTable, 
SLOT(resizeLastRow()));
        connect(unicodeButton, SIGNAL(chosenUnicode(const QString &)), 
m_userTableModel, SLOT(appendUnicode(const QString &)));
-       connect(hideButton, SIGNAL(toggled(bool)), this, 
SLOT(hideButton_toggled(bool)));
+       connect(enhancedDialogButton, SIGNAL(toggled(bool)), this, 
SLOT(enhancedDialogButton_toggled(bool)));
        connect(this, SIGNAL(insertUserSpecialChar(QChar, QString)), this, 
SLOT(slot_insertUserSpecialChar(QChar, QString)));
        connect(uniLoadButton, SIGNAL(clicked()), this, 
SLOT(uniLoadButton_clicked()));
        connect(uniSaveButton, SIGNAL(clicked()), this, 
SLOT(uniSaveButton_clicked()));
@@ -193,7 +193,7 @@
        QApplication::changeOverrideCursor(QCursor(Qt::WaitCursor));
        m_enhanced = new CharSelectEnhanced(this);
        connect(m_enhanced, SIGNAL(insertSpecialChars(const QVector<uint> &)), 
this, SLOT(slot_insertSpecialChars(const QVector<uint> &)));
-       connect(m_enhanced, SIGNAL(paletteShown(bool)), hideButton, 
SLOT(setChecked(bool)));
+       connect(m_enhanced, SIGNAL(paletteShown(bool)), enhancedDialogButton, 
SLOT(setChecked(bool)));
        m_enhanced->setDoc(m_doc);
        m_enhanced->setEnabled(this->isEnabled());
        m_enhanced->show();
@@ -205,20 +205,20 @@
        if (!m_enhanced)
                return;
 
-       hideButton->blockSignals(true);
-       hideButton->setChecked(false);
-       hideButton->blockSignals(false);
+       enhancedDialogButton->blockSignals(true);
+       enhancedDialogButton->setChecked(false);
+       enhancedDialogButton->blockSignals(false);
 
        disconnect(m_enhanced, SIGNAL(insertSpecialChars(const QVector<uint> 
&)), this, SLOT(slot_insertSpecialChars(const QVector<uint> &)));
-       disconnect(m_enhanced, SIGNAL(paletteShown(bool)), hideButton, 
SLOT(setChecked(bool)));
+       disconnect(m_enhanced, SIGNAL(paletteShown(bool)), 
enhancedDialogButton, SLOT(setChecked(bool)));
        m_enhanced->close();
        delete m_enhanced;
        m_enhanced = 0;
 }
 
-void CharSelect::hideButton_toggled(bool state)
-{
-//     tDebug("hideButton_toggled start");
+void CharSelect::enhancedDialogButton_toggled(bool state)
+{
+//     tDebug("enhancedDialogButton_toggled start");
        if (!m_doc)
                return;
 
@@ -227,7 +227,7 @@
        else
                openEnhanced();
 
-//     tDebug("hideButton_toggled end");
+//     tDebug("enhancedDialogButton_toggled end");
 }
 
 void CharSelect::hide()

Modified: trunk/Scribus/scribus/ui/charselect.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/ui/charselect.h
==============================================================================
--- trunk/Scribus/scribus/ui/charselect.h       (original)
+++ trunk/Scribus/scribus/ui/charselect.h       Mon Dec 12 22:14:13 2016
@@ -83,15 +83,16 @@
        void closeEnhanced();
 
 private slots:
-       void userNewChar(uint i, QString font);
-       void hideButton_toggled(bool);
+       void enhancedDialogButton_toggled(bool);
        void slot_insertSpecialChar();
        void slot_insertSpecialChars(const QVector<uint> & chars);
        void slot_insertUserSpecialChar(QChar, QString font);
        void slot_selectionChanged( const QItemSelection&, const 
QItemSelection& );
+       void uniClearButton_clicked();
        void uniLoadButton_clicked();
        void uniSaveButton_clicked();
-       void uniClearButton_clicked();
+       void userNewChar(uint i, QString font);
+
 };
 
 #endif

Modified: trunk/Scribus/scribus/ui/charselect.ui
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/ui/charselect.ui
==============================================================================
--- trunk/Scribus/scribus/ui/charselect.ui      (original)
+++ trunk/Scribus/scribus/ui/charselect.ui      Mon Dec 12 22:14:13 2016
@@ -20,7 +20,7 @@
       <number>1</number>
      </property>
      <item>
-      <widget class="QPushButton" name="hideButton">
+      <widget class="QPushButton" name="enhancedDialogButton">
        <property name="toolTip">
         <string>Show/Hide Enhanced Palette</string>
        </property>
@@ -103,7 +103,7 @@
   </customwidget>
  </customwidgets>
  <tabstops>
-  <tabstop>hideButton</tabstop>
+  <tabstop>enhancedDialogButton</tabstop>
   <tabstop>unicodeButton</tabstop>
   <tabstop>uniLoadButton</tabstop>
   <tabstop>uniSaveButton</tabstop>

Modified: trunk/Scribus/scribus/ui/charselectenhanced.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/ui/charselectenhanced.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/charselectenhanced.cpp     (original)
+++ trunk/Scribus/scribus/ui/charselectenhanced.cpp     Mon Dec 12 22:14:13 2016
@@ -37,8 +37,7 @@
        connect(m_charTable, SIGNAL(selectChar(uint, QString)), this, 
SLOT(newChar(uint, QString)));
        connect(fontSelector, SIGNAL(activated(int)), this, SLOT(newFont(int)));
        connect(rangeSelector, SIGNAL(activated(int)), this, 
SLOT(newCharClass(int)));
-       connect(hexLineEdit, SIGNAL(returnPressed()),
-                       this, SLOT(hexLineEdit_returnPressed()));
+       connect(hexLineEdit, SIGNAL(returnPressed()), this, 
SLOT(hexLineEdit_returnPressed()));
 }
 
 CharSelectEnhanced::~CharSelectEnhanced()

Modified: trunk/Scribus/scribus/ui/propertywidget_distance.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=21646&path=/trunk/Scribus/scribus/ui/propertywidget_distance.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/propertywidget_distance.cpp        (original)
+++ trunk/Scribus/scribus/ui/propertywidget_distance.cpp        Mon Dec 12 
22:14:13 2016
@@ -132,7 +132,7 @@
        if (columnGapLabel->currentIndex() == 0)
        {
                columnGap->setMaximum(qMax((textItem->width() / textItem->Cols 
- textItem->textToFrameDistLeft() - textItem->textToFrameDistRight()) * 
m_unitRatio, 0.0));
-               columnGap->setValue(textItem->ColGap*m_unitRatio);
+               columnGap->setValue(textItem->ColGap * m_unitRatio);
        }
        else
        {
@@ -318,17 +318,16 @@
        PageItem *textItem = m_item;
        if (m_doc->appMode == modeEditTable)
                textItem = m_item->asTable()->activeCell().textFrame();
-       if (!textItem) return;
+       if (!textItem)
+               return;
 
        if (columnGapLabel->currentIndex() == 0)
                textItem->setColumnGap(columnGap->value() / m_unitRatio);
        else
        {
-               double lineCorr;
+               double lineCorr=0.0;
                if ((textItem->lineColor() != CommonStrings::None) || 
(!textItem->strokePattern().isEmpty()))
                        lineCorr = textItem->lineWidth();
-               else
-                       lineCorr = 0;
                double newWidth = columnGap->value() / m_unitRatio;
                double newGap = qMax(((textItem->width() - 
textItem->textToFrameDistLeft() - textItem->textToFrameDistRight() - lineCorr) 
- (newWidth * textItem->Cols)) / (textItem->Cols - 1), 0.0);
                textItem->setColumnGap(newGap);


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

Reply via email to