Rosanne DiMesio : Test report form formatting changes

Jeremy Newman jnewman at winehq.org
Mon Aug 14 10:57:16 CDT 2017


Module: appdb
Branch: master
Commit: cae08624a90f98493b4a3d765a922680eb990064
URL:    http://source.winehq.org/git/appdb.git/?a=commit;h=cae08624a90f98493b4a3d765a922680eb990064

Author: Rosanne DiMesio <dimesio at earthlink.net>
Date:   Fri Aug 11 15:14:00 2017 -0500

Test report form formatting changes

Moves the Installs? and Runs? questions to the top of the form.
Uses CSS to control the table stripes instead of setting each
row's color manually.

Signed-off-by: Rosanne DiMesio <dimesio at earthlink.net>
Signed-off-by: Jeremy Newman <jnewman at codeweavers.com>

---

 css/styles.css       |  7 ++++++
 include/testData.php | 66 ++++++++++++++++++++++++++++------------------------
 2 files changed, 43 insertions(+), 30 deletions(-)

diff --git a/css/styles.css b/css/styles.css
index 0ff3e35..adfd3b4 100644
--- a/css/styles.css
+++ b/css/styles.css
@@ -663,6 +663,13 @@ table.whq-table tbody tr td {
 table.whq-table tbody tr:nth-child(odd) td {
     background-color: #ddd;
 }
+/* used for the test report form */
+table.whq-table-striped tbody tr:nth-child(odd) td {
+   background-color: #E0E0E0; 
+}
+table.whq-table-striped tbody tr:nth-child(even) td {
+   background-color: #C0C0C0; 
+}
 
 /* box */
 .box-title {
diff --git a/include/testData.php b/include/testData.php
index d1af688..38763c2 100644
--- a/include/testData.php
+++ b/include/testData.php
@@ -789,27 +789,41 @@ class testData{
         $sName = version::fullName($this->iVersionId);
 
         echo html_frame_start("Test Form - $sName", "90%", "", 0);
-        echo "<table width='100%' border=0 cellpadding=2 cellspacing=0>\n";
+        
+        echo '<table class="whq-table whq-table-striped table-bordered" width="100%" border=0 cellpadding=2 cellspacing=0>';
+        
+        // Installs
+        echo '<tr><td><b>Installs?</b></td><td>',"\n";
+        testData::make_Installs_list("sInstalls", $this->sInstalls);
+        echo '  Installing is an important part of testing under Wine. Select N/A if there is no installer.</td></tr>',"\n";
+       
+        // Runs
+        echo '<tr><td><b>Runs?</b></td><td>',"\n";
+        testData::make_Runs_list("sRuns", $this->sRuns);
+        echo '</td></tr>',"\n";
 
         // What works
-        echo '<tr valign=top><td class="color0"><b>What works</b></td>',"\n";
-        echo '<td class="color0"><p><textarea cols="80" rows="20" id="Test1" name="shWhatWorks" class="wysiwyg">';
+        echo '<tr valign=top><td><b>What works</b></td>',"\n";
+        echo '<td><p><textarea cols="80" rows="20" id="Test1" name="shWhatWorks" class="wysiwyg">';
         echo $this->shWhatWorks.'</textarea></p></td></tr>',"\n";
+        
         // What Does not work
-        echo '<tr valign=top><td class=color1><b>What does not work</b></td>',"\n";
-        echo '<td class="color0"><p><textarea cols="80" rows="20" id="Test2" name="shWhatDoesnt" class="wysiwyg">';
+        echo '<tr valign=top><td><b>What does not work</b></td>',"\n";
+        echo '<td><p><textarea cols="80" rows="20" id="Test2" name="shWhatDoesnt" class="wysiwyg">';
         echo $this->shWhatDoesnt.'</textarea></p></td></tr>',"\n";
+        
         // What was not tested
-        echo '<tr valign=top><td class=color0><b>What was not tested</b></td>',"\n";
-        echo '<td class="color0"><p><textarea cols="80" rows="20" id="Test3" name="shWhatNotTested" class="wysiwyg">';
+        echo '<tr valign=top><td><b>What was not tested</b></td>',"\n";
+        echo '<td><p><textarea cols="80" rows="20" id="Test3" name="shWhatNotTested" class="wysiwyg">';
         echo $this->shWhatNotTested.'</textarea></p></td></tr>',"\n";
+        
         // Date Tested
-        echo '<tr valign=top><td class="color1"><b>Date tested </b></td>',"\n";
-        echo '<td class="color0"><input type=text name="sTestedDate" value="'.$this->sTestedDate.'" size="20"></td></tr>',"\n";
-        echo '<tr valign=top><td class="color1"></td><td class="color0"><p/>YYYY-MM-DD HH:MM:SS</td></tr>',"\n";
+        echo '<tr valign=top><td><b>Date tested </b></td>',"\n";
+        echo '<td><input type=text name="sTestedDate" value="'.$this->sTestedDate.'" size="20"><br>';
+        echo 'YYYY-MM-DD HH:MM:SS</td></tr>',"\n";
 
         // Version List
-        echo '<tr><td class=color1><b>Tested release</b></td><td class=color0>',"\n";
+        echo '<tr><td><b>Tested release</b></td><td>',"\n";
         echo make_bugzilla_version_list("sTestedRelease", $this->sTestedRelease);
         // Give the user some information about our available versions
         echo "<span>Version not listed?  Your Wine is too old, <a href=\"//winehq.org/download\">upgrade!</a></span><br>";
@@ -822,28 +836,16 @@ class testData{
         echo '  Wine-staging';
         echo '</label>';
         echo '</td></tr>',"\n";
-
-        // Installs
-        echo '<tr><td class=color0><b>Installs?</b></td><td class=color0>',"\n";
-        testData::make_Installs_list("sInstalls", $this->sInstalls);
-        echo '  Installing is an important part of testing under Wine. Select N/A if there is no installer.</td></tr>',"\n";
-        // Runs
-        echo '<tr><td class=color1><b>Runs?</b></td><td class=color0>',"\n";
-        testData::make_Runs_list("sRuns", $this->sRuns);
-        echo '</td></tr>',"\n";
+       
         // Rating
-        echo '<tr><td class="color0"><b>Rating</b></td><td class="color0">',"\n";
+        echo '<tr><td><b>Rating</b></td><td>',"\n";
         echo make_maintainer_rating_list("sTestedRating", $this->sTestedRating);
         echo '<a href="https://wiki.winehq.org/AppDB_Maintainer_Rating_Definitions" target="_blank">Rating definitions</a></td></tr>',"\n";
         // extra comments
-        echo '<tr valign=top><td class="color1"><b>Extra comments</b></td>',"\n";
-        echo '<td class="color0"><textarea name="sComments" id="extra_comments" rows=10 cols=65>';
+        echo '<tr valign=top><td><b>Extra comments</b></td>',"\n";
+        echo '<td><textarea name="sComments" id="extra_comments" rows=10 cols=65>';
         echo $this->sComments.'</textarea></td></tr>',"\n";
 
-        echo '<input type="hidden" name="iVersionId" value="'.$this->iVersionId.'" >';
-        echo '<input type="hidden" name="iTestingId" value="'.$this->iTestingId.'" >';
-        echo '<input type="hidden" name="iTestDataId" value="'.$this->iTestingId.'" >';
-
         // Distribution
         $oDistribution = new distribution($this->iDistributionId);
         $sDistributionHelp = "";
@@ -859,9 +861,9 @@ class testData{
             }
         }
 
-        echo '<tr valign=top><td class="color0"><b>Operating system</b></td class="color0">',"\n";
+        echo '<tr valign=top><td><b>Operating system</b></td>',"\n";
 
-        echo '<td class=color0>',"\n";
+        echo '<td>',"\n";
         echo $sDistributionHelp;
         distribution::make_distribution_list("iTestDistributionId", $this->iDistributionId);
         echo '</td></tr>',"\n";
@@ -873,7 +875,7 @@ class testData{
         {
             if(getInput('bConfirmTestedVersionChange', $aClean) != 'true')
             {
-                echo '<tr><td class="color1"> </td><td class="color0">';
+                echo '<tr><td> </td><td>';
                 echo 'You have changed the Wine version of the report.  Are you sure you want to do this?  Please submit a new test report for every Wine version you test; this is useful for tracking Wine\'s progress.<br>';
                 echo '<input type="checkbox" name="bConfirmTestedVersionChange" value="true"> ';
                 echo 'Yes, I want to change the Wine version';
@@ -884,6 +886,10 @@ class testData{
             }
         }
 
+        echo '<input type="hidden" name="iVersionId" value="'.$this->iVersionId.'" >';
+        echo '<input type="hidden" name="iTestingId" value="'.$this->iTestingId.'" >';
+        echo '<input type="hidden" name="iTestDataId" value="'.$this->iTestingId.'" >';
+        
         echo "</table>\n";
 
         echo html_frame_end();




More information about the wine-cvs mailing list