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
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-04-09 19:16:06 +0000
3+++ debian/changelog 2012-04-09 21:24:23 +0000
4@@ -9,7 +9,13 @@
5 * qt/frontend/qtfront.cpp: Submit/View results buttons are disabled until
6 every selected test has been run (LP: #937715)
7
8- -- Jeff Lane <jeff@ubuntu.com> Mon, 09 Apr 2012 15:15:01 -0400
9+ [Jeff Lane]
10+ * Converted submissionWarningLabel and text to submissionUbuntuFriendlyLabel
11+ wtih instructional text for submitting results. This is a workaround for
12+ the bug causing the warning to be displayed at all times rather than only
13+ when testing is incomplete. (LP: #967457)
14+
15+ -- Jeff Lane <jeff@ubuntu.com> Mon, 09 Apr 2012 17:12:51 -0400
16
17 checkbox (0.13.6) precise; urgency=low
18
19
20=== modified file 'qt/checkbox-qt.ui'
21--- qt/checkbox-qt.ui 2012-02-14 22:10:11 +0000
22+++ qt/checkbox-qt.ui 2012-04-09 21:24:23 +0000
23@@ -825,8 +825,8 @@
24 &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;
25 p, li { white-space: pre-wrap; }
26 &lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Ubuntu'; font-size:10pt; font-weight:400; font-style:normal;&quot;&gt;
27-&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;
28-&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>
29+&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;
30+&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>
31 </property>
32 <property name="scaledContents">
33 <bool>false</bool>
34
35=== modified file 'qt/frontend/qtfront.cpp'
36--- qt/frontend/qtfront.cpp 2012-04-04 14:15:25 +0000
37+++ qt/frontend/qtfront.cpp 2012-04-09 21:24:23 +0000
38@@ -241,7 +241,7 @@
39 m_mainWindow->show();
40 ui->welcomeTextBox->setPlainText(text);
41 } else if (currentState == SUBMISSION) {
42- ui->submissionWarningLabel->setText(text);
43+ ui->submissionUbuntuFriendlyLabel->setText(text);
44 }
45 }
46
47
48=== modified file 'qt/frontend/qtfront.ui'
49--- qt/frontend/qtfront.ui 2012-03-20 16:57:40 +0000
50+++ qt/frontend/qtfront.ui 2012-04-09 21:24:23 +0000
51@@ -1010,7 +1010,7 @@
52 <attribute name="title">
53 <string> Results </string>
54 </attribute>
55- <widget class="QLabel" name="submissionWarningLabel">
56+ <widget class="QLabel" name="submissionUbuntuFriendlyLabel">
57 <property name="geometry">
58 <rect>
59 <x>30</x>
60@@ -1032,8 +1032,8 @@
61 &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;
62 p, li { white-space: pre-wrap; }
63 &lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Ubuntu'; font-size:10pt; font-weight:400; font-style:normal;&quot;&gt;
64-&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;
65-&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>
66+&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;
67+&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>
68 </property>
69 <property name="scaledContents">
70 <bool>false</bool>

Subscribers

People subscribed via source and target branches