View Issue Details

IDProjectCategoryView StatusLast Update
0016387ScribusUser Interfacepublic2021-03-21 15:29
Reporterbubu Assigned Tojghali  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.5.7.svn 
Fixed in Version1.5.7.svn 
Summary0016387: Remove unnecessary group boxes from Insert a frame interface
DescriptionThis issue/patch is similar to 0005781:0016364: In "Insert > Insert a frame" there is 4 tabs and in the "Type" and the "Size" tabs there is an unnamed QGroupBox which causes a blank line inside the group box.

In this patch I propose to remove this blank line from the top of the group box and replace the QGroupBox with a simple QVBoxLayout layout. In first case I simple removed the group box because there was already a box layout and in second case I replaced box group with a box layout. Please see the attached before-after screenshot.

Please, review the patch. The patch seems big but except these changes there are only whitespace modifications. Thanks.
TagsNo tags attached.
PatchYes

Relationships

child of 0013906 acknowledged Metabug: UI enhancement proposals (1.5.x.svn) 

Activities

bubu

2020-12-14 07:06

reporter  

insert-a-frame.diff (12,277 bytes)   
diff --git a/scribus/ui/insertaframe.ui b/scribus/ui/insertaframe.ui
index 18c87cf554b964eb79cf61cf4a70fd5227b7f99a..9dfab66bb56e36baff05251536da3e1142d3db3a 100644
--- a/scribus/ui/insertaframe.ui
+++ b/scribus/ui/insertaframe.ui
@@ -7,7 +7,7 @@
     <x>0</x>
     <y>0</y>
     <width>467</width>
-    <height>364</height>
+    <height>328</height>
    </rect>
   </property>
   <property name="windowTitle">
@@ -25,37 +25,25 @@
       </attribute>
       <layout class="QHBoxLayout">
        <item>
-        <layout class="QVBoxLayout">
+        <layout class="QVBoxLayout" name="typeGroupBox">
          <item>
-          <widget class="QGroupBox" name="typeGroupBox">
+          <widget class="QRadioButton" name="radioButtonTextFrame">
            <property name="toolTip">
-            <string/>
+            <string>Insert one or more text frames</string>
            </property>
-           <property name="title">
-            <string/>
+           <property name="text">
+            <string>&amp;Text Frame</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <widget class="QRadioButton" name="radioButtonImageFrame">
+           <property name="toolTip">
+            <string>Insert one or more image frames</string>
+           </property>
+           <property name="text">
+            <string>&amp;Image Frame</string>
            </property>
-           <layout class="QVBoxLayout">
-            <item>
-             <widget class="QRadioButton" name="radioButtonTextFrame">
-              <property name="toolTip">
-               <string>Insert one or more text frames</string>
-              </property>
-              <property name="text">
-               <string>&amp;Text Frame</string>
-              </property>
-             </widget>
-            </item>
-            <item>
-             <widget class="QRadioButton" name="radioButtonImageFrame">
-              <property name="toolTip">
-               <string>Insert one or more image frames</string>
-              </property>
-              <property name="text">
-               <string>&amp;Image Frame</string>
-              </property>
-             </widget>
-            </item>
-           </layout>
           </widget>
          </item>
          <item>
@@ -301,144 +289,148 @@
       </attribute>
       <layout class="QHBoxLayout">
        <item>
-        <widget class="QGroupBox" name="sizeGroupBox">
-         <property name="toolTip">
-          <string/>
+        <layout class="QVBoxLayout" name="sizeGroupBox">
+         <property name="leftMargin">
+          <number>0</number>
          </property>
-         <property name="title">
-          <string/>
+         <property name="topMargin">
+          <number>0</number>
          </property>
-         <layout class="QVBoxLayout">
-          <item>
-           <widget class="QRadioButton" name="radioButtonPageMarginSize">
-            <property name="toolTip">
-             <string>Insert the new frame with the same dimensions as the page margins</string>
-            </property>
-            <property name="text">
-             <string>Same as the Page Margins</string>
-            </property>
-           </widget>
-          </item>
-          <item>
-           <widget class="QRadioButton" name="radioButtonPageSize">
-            <property name="toolTip">
-             <string>Insert the new frame with the same dimensions as the page</string>
-            </property>
-            <property name="text">
-             <string>Same as the Page</string>
-            </property>
-           </widget>
-          </item>
-          <item>
-           <widget class="QRadioButton" name="radioButtonBleedSize">
-            <property name="toolTip">
-             <string>Insert the new frame with the same dimensions as the bleed area outside the boundary of the page itself</string>
-            </property>
-            <property name="text">
-             <string>Same as the Bleed</string>
-            </property>
-           </widget>
-          </item>
-          <item>
-           <widget class="QRadioButton" name="radioButtonImageSize">
-            <property name="toolTip">
-             <string>Insert the new frame with the same dimensions as the image that will be imported</string>
-            </property>
-            <property name="text">
-             <string>Same as the Imported Image</string>
-            </property>
-           </widget>
-          </item>
-          <item>
-           <widget class="QRadioButton" name="radioButtonCustomSize">
-            <property name="toolTip">
-             <string>Insert the new frame with a custom size</string>
-            </property>
-            <property name="text">
-             <string>Custom Size</string>
-            </property>
-           </widget>
-          </item>
-          <item>
-           <layout class="QHBoxLayout">
-            <item>
-             <spacer>
-              <property name="orientation">
-               <enum>Qt::Horizontal</enum>
-              </property>
-              <property name="sizeType">
-               <enum>QSizePolicy::Fixed</enum>
-              </property>
-              <property name="sizeHint" stdset="0">
-               <size>
-                <width>30</width>
-                <height>20</height>
-               </size>
-              </property>
-             </spacer>
-            </item>
-            <item>
-             <layout class="QGridLayout">
-              <item row="1" column="0">
-               <widget class="QLabel" name="label_3">
-                <property name="text">
-                 <string>Height:</string>
-                </property>
-               </widget>
-              </item>
-              <item row="0" column="0">
-               <widget class="QLabel" name="label_4">
-                <property name="text">
-                 <string>Width:</string>
-                </property>
-               </widget>
-              </item>
-              <item row="0" column="1">
-               <widget class="ScrSpinBox" name="widthScrSpinBox">
-                <property name="toolTip">
-                 <string>Width of the inserted frame</string>
-                </property>
-               </widget>
-              </item>
-              <item row="1" column="1">
-               <widget class="ScrSpinBox" name="heightScrSpinBox">
-                <property name="toolTip">
-                 <string>Height of the inserted frame</string>
-                </property>
-               </widget>
-              </item>
-             </layout>
-            </item>
-            <item>
-             <spacer>
-              <property name="orientation">
-               <enum>Qt::Horizontal</enum>
-              </property>
-              <property name="sizeHint" stdset="0">
-               <size>
-                <width>40</width>
-                <height>20</height>
-               </size>
-              </property>
-             </spacer>
-            </item>
-           </layout>
-          </item>
-          <item>
-           <spacer>
-            <property name="orientation">
-             <enum>Qt::Vertical</enum>
-            </property>
-            <property name="sizeHint" stdset="0">
-             <size>
-              <width>20</width>
-              <height>40</height>
-             </size>
-            </property>
-           </spacer>
-          </item>
-         </layout>
-        </widget>
+         <property name="rightMargin">
+          <number>0</number>
+         </property>
+         <property name="bottomMargin">
+          <number>0</number>
+         </property>
+         <item>
+          <widget class="QRadioButton" name="radioButtonPageMarginSize">
+           <property name="toolTip">
+            <string>Insert the new frame with the same dimensions as the page margins</string>
+           </property>
+           <property name="text">
+            <string>Same as the Page Margins</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <widget class="QRadioButton" name="radioButtonPageSize">
+           <property name="toolTip">
+            <string>Insert the new frame with the same dimensions as the page</string>
+           </property>
+           <property name="text">
+            <string>Same as the Page</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <widget class="QRadioButton" name="radioButtonBleedSize">
+           <property name="toolTip">
+            <string>Insert the new frame with the same dimensions as the bleed area outside the boundary of the page itself</string>
+           </property>
+           <property name="text">
+            <string>Same as the Bleed</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <widget class="QRadioButton" name="radioButtonImageSize">
+           <property name="toolTip">
+            <string>Insert the new frame with the same dimensions as the image that will be imported</string>
+           </property>
+           <property name="text">
+            <string>Same as the Imported Image</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <widget class="QRadioButton" name="radioButtonCustomSize">
+           <property name="toolTip">
+            <string>Insert the new frame with a custom size</string>
+           </property>
+           <property name="text">
+            <string>Custom Size</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <layout class="QHBoxLayout">
+           <item>
+            <spacer>
+             <property name="orientation">
+              <enum>Qt::Horizontal</enum>
+             </property>
+             <property name="sizeType">
+              <enum>QSizePolicy::Fixed</enum>
+             </property>
+             <property name="sizeHint" stdset="0">
+              <size>
+               <width>30</width>
+               <height>20</height>
+              </size>
+             </property>
+            </spacer>
+           </item>
+           <item>
+            <layout class="QGridLayout">
+             <item row="1" column="0">
+              <widget class="QLabel" name="label_3">
+               <property name="text">
+                <string>Height:</string>
+               </property>
+              </widget>
+             </item>
+             <item row="0" column="0">
+              <widget class="QLabel" name="label_4">
+               <property name="text">
+                <string>Width:</string>
+               </property>
+              </widget>
+             </item>
+             <item row="0" column="1">
+              <widget class="ScrSpinBox" name="widthScrSpinBox">
+               <property name="toolTip">
+                <string>Width of the inserted frame</string>
+               </property>
+              </widget>
+             </item>
+             <item row="1" column="1">
+              <widget class="ScrSpinBox" name="heightScrSpinBox">
+               <property name="toolTip">
+                <string>Height of the inserted frame</string>
+               </property>
+              </widget>
+             </item>
+            </layout>
+           </item>
+           <item>
+            <spacer>
+             <property name="orientation">
+              <enum>Qt::Horizontal</enum>
+             </property>
+             <property name="sizeHint" stdset="0">
+              <size>
+               <width>40</width>
+               <height>20</height>
+              </size>
+             </property>
+            </spacer>
+           </item>
+          </layout>
+         </item>
+         <item>
+          <spacer>
+           <property name="orientation">
+            <enum>Qt::Vertical</enum>
+           </property>
+           <property name="sizeHint" stdset="0">
+            <size>
+             <width>20</width>
+             <height>40</height>
+            </size>
+           </property>
+          </spacer>
+         </item>
+        </layout>
        </item>
       </layout>
      </widget>
insert-a-frame.diff (12,277 bytes)   
insert-a-frame.gif (74,607 bytes)   
insert-a-frame.gif (74,607 bytes)   

jghali

2020-12-14 22:13

administrator   ~0048554

Applied, thanks!

FirasH

2021-03-21 15:29

developer   ~0049029

Thanks!
Fixed in 1.5.7.svn (24324)

Issue History

Date Modified Username Field Change
2020-12-14 07:06 bubu New Issue
2020-12-14 07:06 bubu File Added: insert-a-frame.diff
2020-12-14 07:06 bubu File Added: insert-a-frame.gif
2020-12-14 07:25 PeterBenedek Relationship added child of 0013906
2020-12-14 07:51 PeterBenedek Summary Remove unnecessary group boxes from Insert a frame interface => [Patch] Remove unnecessary group boxes from Insert a frame interface
2020-12-14 22:13 jghali Summary [Patch] Remove unnecessary group boxes from Insert a frame interface => Remove unnecessary group boxes from Insert a frame interface
2020-12-14 22:13 jghali Assigned To => jghali
2020-12-14 22:13 jghali Status new => resolved
2020-12-14 22:13 jghali Resolution open => fixed
2020-12-14 22:13 jghali Fixed in Version => 1.5.7.svn
2020-12-14 22:13 jghali Note Added: 0048554
2021-03-21 15:29 FirasH Status resolved => closed
2021-03-21 15:29 FirasH Note Added: 0049029