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