Merge lp:~bladernr/checkbox/967457-remove-warning into lp:checkbox

Proposed by Jeff Lane 
Status: Merged
Merged at revision: 1355
Proposed branch: lp:~bladernr/checkbox/967457-remove-warning
Merge into: lp:checkbox
Diff against target: 70 lines (+13/-7)
4 files modified
debian/changelog (+7/-1)
qt/checkbox-qt.ui (+2/-2)
qt/frontend/qtfront.cpp (+1/-1)
qt/frontend/qtfront.ui (+3/-3)
To merge this branch: bzr merge lp:~bladernr/checkbox/967457-remove-warning
Reviewer Review Type Date Requested Status
Checkbox Developers Pending
Review via email: mp+101293@code.launchpad.net

Description of the change

Per discussion with Ara, as a workaround for bug 967457, this branch converts submissionWarningLabel to submissionUbuntuFriendlyLabel and changes the label text to provide instructions on submitting results to Ubuntu Friendly.

I toyed initially on simply not displaying the label at all, and removing the label from the ui files and cpp file, but in doing that, the submission page looked pretty sparse and "unfriendly". This is a good compromise until we can add some bits in later that will show the warning message only when testing is incomplete.

Also, the warning message is a bit unnecessary before the end of testing because Sylvain Pineau's changes for bug #937715 make the submit and view buttons disabled until testing is complete anyway.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-04-09 19:16:06 +0000
+++ debian/changelog 2012-04-09 21:24:23 +0000
@@ -9,7 +9,13 @@
9 * qt/frontend/qtfront.cpp: Submit/View results buttons are disabled until9 * qt/frontend/qtfront.cpp: Submit/View results buttons are disabled until
10 every selected test has been run (LP: #937715)10 every selected test has been run (LP: #937715)
1111
12 -- Jeff Lane <jeff@ubuntu.com> Mon, 09 Apr 2012 15:15:01 -040012 [Jeff Lane]
13 * Converted submissionWarningLabel and text to submissionUbuntuFriendlyLabel
14 wtih instructional text for submitting results. This is a workaround for
15 the bug causing the warning to be displayed at all times rather than only
16 when testing is incomplete. (LP: #967457)
17
18 -- Jeff Lane <jeff@ubuntu.com> Mon, 09 Apr 2012 17:12:51 -0400
1319
14checkbox (0.13.6) precise; urgency=low20checkbox (0.13.6) precise; urgency=low
1521
1622
=== modified file 'qt/checkbox-qt.ui'
--- qt/checkbox-qt.ui 2012-02-14 22:10:11 +0000
+++ qt/checkbox-qt.ui 2012-04-09 21:24:23 +0000
@@ -825,8 +825,8 @@
825&lt;html&gt;&lt;head&gt;&lt;meta name=&quot;qrichtext&quot; content=&quot;1&quot; /&gt;&lt;style type=&quot;text/css&quot;&gt;825&lt;html&gt;&lt;head&gt;&lt;meta name=&quot;qrichtext&quot; content=&quot;1&quot; /&gt;&lt;style type=&quot;text/css&quot;&gt;
826p, li { white-space: pre-wrap; }826p, li { white-space: pre-wrap; }
827&lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Ubuntu'; font-size:10pt; font-weight:400; font-style:normal;&quot;&gt;827&lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Ubuntu'; font-size:10pt; font-weight:400; font-style:normal;&quot;&gt;
828&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#ff0000;&quot;&gt;Warning: Not all tests have been run yet.&lt;/span&gt;&lt;/p&gt;828&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#000000;&quot;&gt;Please enter the e-mail address associated with your Launchpad account &#40;if applicable&#41;&lt;/span&gt;&lt;/p&gt;
829&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#ff0000;&quot;&gt;You can send the results now, but the submission won't make it to Ubuntu Friendly.&lt;/span&gt;&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>829&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#000000;&quot;&gt;and click the Submit Results button to submit these test results to Ubuntu Friendly.&lt;/span&gt;&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
830 </property>830 </property>
831 <property name="scaledContents">831 <property name="scaledContents">
832 <bool>false</bool>832 <bool>false</bool>
833833
=== modified file 'qt/frontend/qtfront.cpp'
--- qt/frontend/qtfront.cpp 2012-04-04 14:15:25 +0000
+++ qt/frontend/qtfront.cpp 2012-04-09 21:24:23 +0000
@@ -241,7 +241,7 @@
241 m_mainWindow->show();241 m_mainWindow->show();
242 ui->welcomeTextBox->setPlainText(text);242 ui->welcomeTextBox->setPlainText(text);
243 } else if (currentState == SUBMISSION) {243 } else if (currentState == SUBMISSION) {
244 ui->submissionWarningLabel->setText(text);244 ui->submissionUbuntuFriendlyLabel->setText(text);
245 }245 }
246}246}
247247
248248
=== modified file 'qt/frontend/qtfront.ui'
--- qt/frontend/qtfront.ui 2012-03-20 16:57:40 +0000
+++ qt/frontend/qtfront.ui 2012-04-09 21:24:23 +0000
@@ -1010,7 +1010,7 @@
1010 <attribute name="title">1010 <attribute name="title">
1011 <string> Results </string>1011 <string> Results </string>
1012 </attribute>1012 </attribute>
1013 <widget class="QLabel" name="submissionWarningLabel">1013 <widget class="QLabel" name="submissionUbuntuFriendlyLabel">
1014 <property name="geometry">1014 <property name="geometry">
1015 <rect>1015 <rect>
1016 <x>30</x>1016 <x>30</x>
@@ -1032,8 +1032,8 @@
1032&lt;html&gt;&lt;head&gt;&lt;meta name=&quot;qrichtext&quot; content=&quot;1&quot; /&gt;&lt;style type=&quot;text/css&quot;&gt;1032&lt;html&gt;&lt;head&gt;&lt;meta name=&quot;qrichtext&quot; content=&quot;1&quot; /&gt;&lt;style type=&quot;text/css&quot;&gt;
1033p, li { white-space: pre-wrap; }1033p, li { white-space: pre-wrap; }
1034&lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Ubuntu'; font-size:10pt; font-weight:400; font-style:normal;&quot;&gt;1034&lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Ubuntu'; font-size:10pt; font-weight:400; font-style:normal;&quot;&gt;
1035&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#ff0000;&quot;&gt;Warning: Not all tests have been run yet.&lt;/span&gt;&lt;/p&gt;1035&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#000000;&quot;&gt;Please enter the e-mail address associated with your Launchpad account &#40;if applicable&#41;&lt;/span&gt;&lt;/p&gt;
1036&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#ff0000;&quot;&gt;You can send the results now, but the submission won't make it to Ubuntu Friendly.&lt;/span&gt;&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>1036&lt;p style=&quot; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; color:#000000;&quot;&gt;and click the Submit Results button to submit these test results to Ubuntu Friendly.&lt;/span&gt;&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
1037 </property>1037 </property>
1038 <property name="scaledContents">1038 <property name="scaledContents">
1039 <bool>false</bool>1039 <bool>false</bool>

Subscribers

People subscribed via source and target branches