From 33e2d3dee383bf8aa67c843466471feda37429b8 Mon Sep 17 00:00:00 2001
From: Jim Evins <evins@snaught.com>
Date: Sun, 12 May 2019 17:30:42 -0400
Subject: [PATCH] Tweaks to PrintView.

 - Increased range of copies spinbox to 100x the number of iterms per page (#51)
 - Fixed issue that blocked user from holding down on spinbox controls
 - Minor layout tweaks
---
 glabels/PrintView.cpp        |  8 ++++--
 glabels/ui/PrintView.ui      | 55 ++++++++++++++++++++++++------------
 glabels/ui/PropertiesView.ui |  6 ++++
 3 files changed, 49 insertions(+), 20 deletions(-)

diff --git a/glabels/PrintView.cpp b/glabels/PrintView.cpp
index a900af3..a6f3f7d 100644
--- a/glabels/PrintView.cpp
+++ b/glabels/PrintView.cpp
@@ -70,6 +70,9 @@ namespace glabels
 
 		connect( mModel, SIGNAL(changed()), this, SLOT(onModelChanged()) );
 
+		copiesSpin->setRange( 1, 100*mModel->frame()->nLabels() );
+		copiesStartSpin->setRange( 1, mModel->frame()->nLabels() );
+
 		onFormChanged();
 	}
 
@@ -79,6 +82,9 @@ namespace glabels
 	///
 	void PrintView::onModelChanged()
 	{
+		copiesSpin->setRange( 1, 100*mModel->frame()->nLabels() );
+		copiesStartSpin->setRange( 1, mModel->frame()->nLabels() );
+
 		updateView();
 	}
 
@@ -88,8 +94,6 @@ namespace glabels
 	///
 	void PrintView::updateView()
 	{
-		copiesStartSpin->setRange( 1, mModel->frame()->nLabels() );
-
 		if ( mRenderer.nPages() == 1 )
 		{
 			if ( mRenderer.nItems() == 1 )
diff --git a/glabels/ui/PrintView.ui b/glabels/ui/PrintView.ui
index d4ea498..b7303ae 100644
--- a/glabels/ui/PrintView.ui
+++ b/glabels/ui/PrintView.ui
@@ -6,7 +6,7 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>759</width>
+    <width>852</width>
     <height>792</height>
    </rect>
   </property>
@@ -73,21 +73,8 @@
        <property name="title">
         <string>Copies</string>
        </property>
-       <layout class="QGridLayout" name="gridLayout_3">
-        <item row="5" column="0">
-         <widget class="QLabel" name="copiesDescriptionLabel">
-          <property name="sizePolicy">
-           <sizepolicy hsizetype="Minimum" vsizetype="Preferred">
-            <horstretch>0</horstretch>
-            <verstretch>0</verstretch>
-           </sizepolicy>
-          </property>
-          <property name="text">
-           <string notr="true">(Will print a total of xx items on nn pages.)</string>
-          </property>
-         </widget>
-        </item>
-        <item row="1" column="0">
+       <layout class="QVBoxLayout" name="verticalLayout_4">
+        <item>
          <layout class="QHBoxLayout" name="horizontalLayout_5">
           <item>
            <widget class="QLabel" name="label_4">
@@ -104,11 +91,20 @@
           </item>
           <item>
            <widget class="QSpinBox" name="copiesSpin">
+            <property name="buttonSymbols">
+             <enum>QAbstractSpinBox::UpDownArrows</enum>
+            </property>
+            <property name="accelerated">
+             <bool>true</bool>
+            </property>
             <property name="minimum">
              <number>1</number>
             </property>
             <property name="maximum">
-             <number>96</number>
+             <number>9999</number>
+            </property>
+            <property name="stepType">
+             <enum>QAbstractSpinBox::DefaultStepType</enum>
             </property>
            </widget>
           </item>
@@ -127,7 +123,7 @@
           </item>
          </layout>
         </item>
-        <item row="2" column="0">
+        <item>
          <layout class="QHBoxLayout" name="horizontalLayout_4">
           <item>
            <widget class="QLabel" name="label_2">
@@ -183,6 +179,29 @@
           </item>
          </layout>
         </item>
+        <item>
+         <layout class="QHBoxLayout" name="horizontalLayout_7">
+          <item>
+           <widget class="QLabel" name="copiesDescriptionLabel">
+            <property name="sizePolicy">
+             <sizepolicy hsizetype="MinimumExpanding" vsizetype="Preferred">
+              <horstretch>0</horstretch>
+              <verstretch>0</verstretch>
+             </sizepolicy>
+            </property>
+            <property name="minimumSize">
+             <size>
+              <width>330</width>
+              <height>0</height>
+             </size>
+            </property>
+            <property name="text">
+             <string notr="true">(Will print a total of xxxx items on nnn pages.)        </string>
+            </property>
+           </widget>
+          </item>
+         </layout>
+        </item>
        </layout>
       </widget>
      </item>
diff --git a/glabels/ui/PropertiesView.ui b/glabels/ui/PropertiesView.ui
index 60c0632..6b8f115 100644
--- a/glabels/ui/PropertiesView.ui
+++ b/glabels/ui/PropertiesView.ui
@@ -241,6 +241,12 @@
         </item>
         <item row="1" column="0">
          <widget class="QPushButton" name="changeProductButton">
+          <property name="minimumSize">
+           <size>
+            <width>330</width>
+            <height>0</height>
+           </size>
+          </property>
           <property name="toolTip">
            <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;Select another product for this gLabels project.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
           </property>