From 957ed9912fba01bd462a1f43e64a73abefdbb7b2 Mon Sep 17 00:00:00 2001
Message-Id: <957ed9912fba01bd462a1f43e64a73abefdbb7b2.1613597688.git.bubu@ujevangelizacio.hu>
From: =?UTF-8?q?Gyuris=20Gell=C3=A9rt?= <bubu@ujevangelizacio.hu>
Date: Wed, 17 Feb 2021 22:33:28 +0100
Subject: [PATCH] Unify margins and spacing for dialogs, set 8

---
 scribus/plugins/colorwheel/cwdialog.ui        | 162 +++---------------
 .../plugins/import/xps/xpsimportoptions.ui    |  15 +-
 scribus/ui/fontreplacedialog.cpp              |   8 +-
 scribus/ui/patternpropsdialog.cpp             |  32 ++--
 scribus/ui/prefs_hyphenatorbase.ui            |  66 ++-----
 5 files changed, 71 insertions(+), 212 deletions(-)

diff --git a/scribus/plugins/colorwheel/cwdialog.ui b/scribus/plugins/colorwheel/cwdialog.ui
index b1962d9d5..74c2a94a9 100644
--- a/scribus/plugins/colorwheel/cwdialog.ui
+++ b/scribus/plugins/colorwheel/cwdialog.ui
@@ -28,9 +28,6 @@
    <property name="bottomMargin">
     <number>9</number>
    </property>
-   <property name="spacing">
-    <number>6</number>
-   </property>
    <item row="0" column="0">
     <widget class="ColorWheel" name="colorWheel" native="true">
      <property name="sizePolicy">
@@ -41,13 +38,13 @@
      </property>
      <property name="minimumSize">
       <size>
-       <width>300</width>
+       <width>307</width>
        <height>300</height>
       </size>
      </property>
      <property name="maximumSize">
       <size>
-       <width>300</width>
+       <width>307</width>
        <height>300</height>
       </size>
      </property>
@@ -62,21 +59,6 @@
       <string>Result Colors</string>
      </property>
      <layout class="QGridLayout">
-      <property name="leftMargin">
-       <number>9</number>
-      </property>
-      <property name="topMargin">
-       <number>9</number>
-      </property>
-      <property name="rightMargin">
-       <number>9</number>
-      </property>
-      <property name="bottomMargin">
-       <number>9</number>
-      </property>
-      <property name="spacing">
-       <number>6</number>
-      </property>
       <item row="1" column="0">
        <widget class="QTableWidget" name="currentColorTable">
         <property name="sizePolicy">
@@ -112,15 +94,15 @@
         <attribute name="horizontalHeaderVisible">
          <bool>false</bool>
         </attribute>
+        <attribute name="horizontalHeaderMinimumSectionSize">
+         <number>42</number>
+        </attribute>
         <attribute name="horizontalHeaderDefaultSectionSize">
          <number>42</number>
         </attribute>
         <attribute name="horizontalHeaderHighlightSections">
          <bool>false</bool>
         </attribute>
-        <attribute name="horizontalHeaderMinimumSectionSize">
-         <number>42</number>
-        </attribute>
         <attribute name="horizontalHeaderStretchLastSection">
          <bool>true</bool>
         </attribute>
@@ -183,26 +165,8 @@
       <string>Color Scheme Method</string>
      </property>
      <layout class="QGridLayout">
-      <property name="leftMargin">
-       <number>9</number>
-      </property>
-      <property name="topMargin">
-       <number>9</number>
-      </property>
-      <property name="rightMargin">
-       <number>9</number>
-      </property>
-      <property name="bottomMargin">
-       <number>9</number>
-      </property>
-      <property name="spacing">
-       <number>6</number>
-      </property>
       <item row="1" column="0">
        <layout class="QHBoxLayout">
-        <property name="spacing">
-         <number>6</number>
-        </property>
         <property name="leftMargin">
          <number>0</number>
         </property>
@@ -264,9 +228,6 @@
    </item>
    <item row="3" column="0" colspan="2">
     <layout class="QHBoxLayout">
-     <property name="spacing">
-      <number>6</number>
-     </property>
      <property name="leftMargin">
       <number>0</number>
      </property>
@@ -342,21 +303,6 @@
       <string>Preview:</string>
      </property>
      <layout class="QGridLayout">
-      <property name="leftMargin">
-       <number>9</number>
-      </property>
-      <property name="topMargin">
-       <number>9</number>
-      </property>
-      <property name="rightMargin">
-       <number>9</number>
-      </property>
-      <property name="bottomMargin">
-       <number>9</number>
-      </property>
-      <property name="spacing">
-       <number>6</number>
-      </property>
       <item row="2" column="0">
        <widget class="QLabel" name="previewLabel">
         <property name="minimumSize">
@@ -432,25 +378,19 @@
       </attribute>
       <layout class="QGridLayout">
        <property name="leftMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="topMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="rightMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="bottomMargin">
-        <number>11</number>
-       </property>
-       <property name="spacing">
-        <number>6</number>
+        <number>9</number>
        </property>
        <item row="0" column="0">
         <layout class="QHBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -487,9 +427,6 @@
        </item>
        <item row="1" column="0">
         <layout class="QHBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -526,9 +463,6 @@
        </item>
        <item row="2" column="0">
         <layout class="QHBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -565,9 +499,6 @@
        </item>
        <item row="3" column="0">
         <layout class="QHBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -604,9 +535,6 @@
        </item>
        <item row="4" column="0">
         <layout class="QHBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -649,9 +577,6 @@
        </item>
        <item row="5" column="0">
         <layout class="QHBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -716,25 +641,19 @@
       </attribute>
       <layout class="QGridLayout">
        <property name="leftMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="topMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="rightMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="bottomMargin">
-        <number>11</number>
-       </property>
-       <property name="spacing">
-        <number>6</number>
+        <number>9</number>
        </property>
        <item row="0" column="0">
         <layout class="QVBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -749,9 +668,6 @@
          </property>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -785,9 +701,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -821,9 +734,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -857,9 +767,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -899,9 +806,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -965,25 +869,19 @@
       </attribute>
       <layout class="QGridLayout">
        <property name="leftMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="topMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="rightMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="bottomMargin">
-        <number>11</number>
-       </property>
-       <property name="spacing">
-        <number>6</number>
+        <number>9</number>
        </property>
        <item row="0" column="0">
         <layout class="QVBoxLayout">
-         <property name="spacing">
-          <number>6</number>
-         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -998,9 +896,6 @@
          </property>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -1034,9 +929,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -1070,9 +962,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -1106,9 +995,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -1151,9 +1037,6 @@
          </item>
          <item>
           <layout class="QHBoxLayout">
-           <property name="spacing">
-            <number>6</number>
-           </property>
            <property name="leftMargin">
             <number>0</number>
            </property>
@@ -1217,19 +1100,16 @@
       </attribute>
       <layout class="QGridLayout">
        <property name="leftMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="topMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="rightMargin">
-        <number>11</number>
+        <number>9</number>
        </property>
        <property name="bottomMargin">
-        <number>11</number>
-       </property>
-       <property name="spacing">
-        <number>6</number>
+        <number>9</number>
        </property>
        <item row="0" column="0">
         <widget class="ColorListBox" name="documentColorList" native="true"/>
diff --git a/scribus/plugins/import/xps/xpsimportoptions.ui b/scribus/plugins/import/xps/xpsimportoptions.ui
index 115ae9020..3ffc58b6a 100644
--- a/scribus/plugins/import/xps/xpsimportoptions.ui
+++ b/scribus/plugins/import/xps/xpsimportoptions.ui
@@ -17,6 +17,18 @@
    <bool>true</bool>
   </property>
   <layout class="QVBoxLayout" name="verticalLayout_4">
+   <property name="leftMargin">
+    <number>9</number>
+   </property>
+   <property name="topMargin">
+    <number>9</number>
+   </property>
+   <property name="rightMargin">
+    <number>9</number>
+   </property>
+   <property name="bottomMargin">
+    <number>9</number>
+   </property>
    <item>
     <layout class="QHBoxLayout" name="horizontalLayout_5">
      <item>
@@ -104,9 +116,6 @@
         </item>
         <item>
          <layout class="QHBoxLayout" name="horizontalLayout">
-          <property name="spacing">
-           <number>3</number>
-          </property>
           <property name="leftMargin">
            <number>15</number>
           </property>
diff --git a/scribus/ui/fontreplacedialog.cpp b/scribus/ui/fontreplacedialog.cpp
index 136a61177..e173fa640 100644
--- a/scribus/ui/fontreplacedialog.cpp
+++ b/scribus/ui/fontreplacedialog.cpp
@@ -31,8 +31,8 @@ FontReplaceDialog::FontReplaceDialog( QWidget* parent, QMap<QString, QString> *R
 	setWindowIcon(IconManager::instance().loadIcon("AppIcon.png"));
 	ReplaceList = RList;
 	FontReplaceDialogLayout = new QVBoxLayout( this );
-	FontReplaceDialogLayout->setMargin(10);
-	FontReplaceDialogLayout->setSpacing(5);
+	FontReplaceDialogLayout->setContentsMargins(9, 9, 9, 9);
+	FontReplaceDialogLayout->setSpacing(6);
 
 	textLabel1 = new QLabel(this);
 	textLabel1->setAlignment(Qt::AlignVCenter);
@@ -66,8 +66,8 @@ FontReplaceDialog::FontReplaceDialog( QWidget* parent, QMap<QString, QString> *R
 	FontReplaceDialogLayout->addWidget( replacementTable );
 
 	layout1 = new QHBoxLayout;
-	layout1->setMargin(0);
-	layout1->setSpacing(5);
+	layout1->setContentsMargins(0, 0, 0, 0);
+	layout1->setSpacing(6);
 	stickyReplacements = new QCheckBox( this );
 	stickyReplacements->setText( tr( "Make these substitutions permanent" ) );
 	layout1->addWidget(stickyReplacements);
diff --git a/scribus/ui/patternpropsdialog.cpp b/scribus/ui/patternpropsdialog.cpp
index 0c5301f0e..c4262ccd1 100644
--- a/scribus/ui/patternpropsdialog.cpp
+++ b/scribus/ui/patternpropsdialog.cpp
@@ -30,13 +30,13 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 	setModal(true);
 	forStroke = isStroke;
 	frame3Layout = new QVBoxLayout( this );
-	frame3Layout->setMargin(0);
-	frame3Layout->setSpacing(2);
+	frame3Layout->setContentsMargins(9, 9, 9, 9);
+	frame3Layout->setSpacing(6);
 
 	groupOffset = new QGroupBox( this );
 	groupOffsetLayout = new QHBoxLayout( groupOffset );
-	groupOffsetLayout->setSpacing( 2 );
-	groupOffsetLayout->setMargin( 3 );
+	groupOffsetLayout->setSpacing(6);
+	groupOffsetLayout->setContentsMargins(9, 9, 9, 9);
 	groupOffsetLayout->setAlignment( Qt::AlignTop );
 	textLabel1 = new QLabel( groupOffset );
 	groupOffsetLayout->addWidget( textLabel1 );
@@ -50,8 +50,8 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 
 	groupScale = new QGroupBox( this );
 	groupScaleLayout = new QGridLayout( groupScale );
-	groupScaleLayout->setSpacing( 2 );
-	groupScaleLayout->setMargin( 3 );
+	groupScaleLayout->setSpacing(6);
+	groupScaleLayout->setContentsMargins(9, 9, 9, 9);
 	groupScaleLayout->setAlignment( Qt::AlignTop );
 	textLabel5 = new QLabel( groupScale );
 	groupScaleLayout->addWidget( textLabel5, 0, 0 );
@@ -71,8 +71,8 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 	frame3Layout->addWidget( groupScale );
 	groupRotation = new QGroupBox( this );
 	groupRotationLayout = new QHBoxLayout( groupRotation );
-	groupRotationLayout->setSpacing( 2 );
-	groupRotationLayout->setMargin( 3 );
+	groupRotationLayout->setSpacing(6);
+	groupRotationLayout->setContentsMargins(9, 9, 9, 9);
 	groupRotationLayout->setAlignment( Qt::AlignTop );
 	textLabel7 = new QLabel( groupRotation );
 	groupRotationLayout->addWidget( textLabel7 );
@@ -82,8 +82,8 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 
 	groupSkew = new QGroupBox( this );
 	groupSkewLayout = new QGridLayout( groupSkew );
-	groupSkewLayout->setSpacing( 2 );
-	groupSkewLayout->setMargin( 3 );
+	groupSkewLayout->setSpacing(6);
+	groupSkewLayout->setContentsMargins(9, 9, 9, 9);
 	groupSkewLayout->setAlignment( Qt::AlignTop );
 	textLabel8 = new QLabel( groupSkew );
 	groupSkewLayout->addWidget( textLabel8, 0, 0 );
@@ -101,8 +101,8 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 	{
 		groupSpace = new QGroupBox( this );
 		groupSpaceLayout = new QHBoxLayout( groupSpace );
-		groupSpaceLayout->setSpacing( 2 );
-		groupSpaceLayout->setMargin( 3 );
+		groupSpaceLayout->setSpacing(6);
+		groupSpaceLayout->setContentsMargins(9, 9, 9, 9);
 		groupSpaceLayout->setAlignment( Qt::AlignTop );
 		textLabel20 = new QLabel( groupSpace );
 		groupSpaceLayout->addWidget( textLabel20 );
@@ -113,8 +113,8 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 	}
 
 	groupFlipLayout = new QHBoxLayout();
-	groupFlipLayout->setSpacing( 2 );
-	groupFlipLayout->setMargin( 3 );
+	groupFlipLayout->setSpacing(6);
+	groupFlipLayout->setContentsMargins(0, 0, 0, 0);
 	textLabel15 = new QLabel( this );
 	groupFlipLayout->addWidget( textLabel15 );
 	QSpacerItem* spacer = new QSpacerItem( 0, 0, QSizePolicy::Expanding, QSizePolicy::Minimum );
@@ -130,8 +130,8 @@ PatternPropsDialog::PatternPropsDialog(QWidget* parent, int unitIndex, bool isSt
 	frame3Layout->addLayout( groupFlipLayout );
 
 	buttonLayout = new QHBoxLayout;
-	buttonLayout->setMargin(0);
-	buttonLayout->setSpacing(5);
+	buttonLayout->setContentsMargins(0, 0, 0, 0);
+	buttonLayout->setSpacing(6);
 	QSpacerItem* hspacing = new QSpacerItem(0, 0, QSizePolicy::Expanding, QSizePolicy::Minimum);
 	buttonLayout->addItem(hspacing);
 	buttonOk = new QPushButton( this );
diff --git a/scribus/ui/prefs_hyphenatorbase.ui b/scribus/ui/prefs_hyphenatorbase.ui
index da3285600..1f6b56827 100644
--- a/scribus/ui/prefs_hyphenatorbase.ui
+++ b/scribus/ui/prefs_hyphenatorbase.ui
@@ -11,24 +11,24 @@
    </rect>
   </property>
   <property name="windowTitle">
-   <string notr="true">Form</string>
-  </property>
-  <layout class="QVBoxLayout" name="verticalLayout">
-   <property name="leftMargin">
-    <number>0</number>
-   </property>
-   <property name="topMargin">
-    <number>0</number>
-   </property>
-   <property name="rightMargin">
-    <number>0</number>
-   </property>
-   <property name="bottomMargin">
-    <number>0</number>
-   </property>
-   <item>
-    <widget class="QLabel" name="label">
-     <property name="font">
+   <string notr="true">Form</string>
+  </property>
+  <layout class="QVBoxLayout" name="verticalLayout">
+   <property name="leftMargin">
+    <number>0</number>
+   </property>
+   <property name="topMargin">
+    <number>0</number>
+   </property>
+   <property name="rightMargin">
+    <number>0</number>
+   </property>
+   <property name="bottomMargin">
+    <number>0</number>
+   </property>
+   <item>
+    <widget class="QLabel" name="label">
+     <property name="font">
       <font>
        <pointsize>14</pointsize>
        <weight>75</weight>
@@ -108,21 +108,6 @@
             <string>Exceptions</string>
            </property>
            <layout class="QGridLayout" name="_2">
-            <property name="leftMargin">
-             <number>10</number>
-            </property>
-            <property name="topMargin">
-             <number>10</number>
-            </property>
-            <property name="rightMargin">
-             <number>10</number>
-            </property>
-            <property name="bottomMargin">
-             <number>10</number>
-            </property>
-            <property name="spacing">
-             <number>5</number>
-            </property>
             <item row="0" column="0" colspan="4">
              <widget class="QListWidget" name="exceptionListWidget"/>
             </item>
@@ -169,21 +154,6 @@
             <string>Ignore List</string>
            </property>
            <layout class="QGridLayout" name="_3">
-            <property name="leftMargin">
-             <number>10</number>
-            </property>
-            <property name="topMargin">
-             <number>10</number>
-            </property>
-            <property name="rightMargin">
-             <number>10</number>
-            </property>
-            <property name="bottomMargin">
-             <number>10</number>
-            </property>
-            <property name="spacing">
-             <number>5</number>
-            </property>
             <item row="0" column="0" colspan="4">
              <widget class="QListWidget" name="ignoreListWidget">
               <property name="sizePolicy">
-- 
2.27.0

