Rosanne DiMesio : Make new distribution added notice more visible

Jeremy Newman jnewman at winehq.org
Thu Jul 13 13:16:41 CDT 2017


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

Author: Rosanne DiMesio <dimesio at earthlink.net>
Date:   Thu Jul 13 11:58:19 2017 -0500

Make new distribution added notice more visible

Add a warning icon and background color to make the inline notice
to maintainers/admins more visible on the form. Also changes one more
instance of distribution to operating system.

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

---

 include/testData.php | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/include/testData.php b/include/testData.php
index 84f87c7..cd3fc8b 100644
--- a/include/testData.php
+++ b/include/testData.php
@@ -811,14 +811,11 @@ class testData{
             if(!$this->iDistributionId)
             {
                 $sDistributionHelp = "If yours is not on the list, ".
-                                     "please add it using the form below.";
+                                     "please add it using the form below.<br>";
             } else
             {
-                $sDistributionHelp = "The user added a new operating system; ".
-                                     "review it in the form below or replace ".
-                                     "it with one from the list.";
+                $sDistributionHelp = '<p class="bg-danger"><span class = "text-danger fa fa-exclamation-triangle" style="font-size:125%"></span> The user submitted a new operating system; please review it in the form below.</p>';
             }
-            $sDistributionHelp .= "<br>\n";
         }
 
         echo '<tr valign=top><td class="color0"><b>Operating system</b></td class="color0">',"\n";
@@ -928,7 +925,7 @@ class testData{
 
         // No Distribution entered, and nothing in the list is selected
         if (empty($aValues['sDistribution']) && !$aValues['iTestDistributionId'])
-            $errors .= "<li>Please enter a distribution.</li>\n";
+            $errors .= "<li>Please enter an operating system.</li>\n";
 
         if (empty($aValues['sInstalls']))
             $errors .= "<li>Please enter whether this application installs or not.</li>\n";




More information about the wine-cvs mailing list