diff --git a/mozilla/webtools/update/css/base/content.css b/mozilla/webtools/update/css/base/content.css index cff8af9c569..8a403a80bb8 100755 --- a/mozilla/webtools/update/css/base/content.css +++ b/mozilla/webtools/update/css/base/content.css @@ -17,6 +17,7 @@ a:hover { color: #333; } a:active { color: #000; } #header a:visited { color: #039; } +#header a:hover { color: #333; } #mBody li { padding-bottom: 0.5em; } diff --git a/mozilla/webtools/update/css/cavendish/content.css b/mozilla/webtools/update/css/cavendish/content.css index 5182e90ee3a..f0700ad776a 100755 --- a/mozilla/webtools/update/css/cavendish/content.css +++ b/mozilla/webtools/update/css/cavendish/content.css @@ -194,4 +194,9 @@ padding-left: 10px; font-size: 8pt; color: #333; +} + +.tooltip { + cursor: help; + border-bottom: 1px dotted; } \ No newline at end of file diff --git a/mozilla/webtools/update/developers/approval.php b/mozilla/webtools/update/developers/approval.php index 0ac6d66dca1..1b0fa390b7a 100755 --- a/mozilla/webtools/update/developers/approval.php +++ b/mozilla/webtools/update/developers/approval.php @@ -63,10 +63,14 @@ if ($type=="T") { if ($approval=="YES") { if ($installation=="YES" and $uninstallation=="YES" and $appworks=="YES" and $cleanprofile=="YES" and $visualerrors=="YES" and $allelementsthemed=="YES" and $testos and $testbuild) { $approval_result = process_approval($type, $file, "approve"); + } else { + echo"Error: Approval cannot be processed because of missing data. Fill in the required fields and try again.
\n"; } } else { if ($testos and $testbuild and $comments) { $approval_result = process_approval($type, $file, "deny"); + } else { + echo"Error: Denial cannot be processed because of missing data. Fill in the required fields and try again.
\n"; } } @@ -75,10 +79,14 @@ if ($type=="T") { if ($approval=="YES") { if ($installation=="YES" and $uninstallation=="YES" and $appworks=="YES" and $cleanprofile=="YES" and $newchrome=="YES" and $worksasdescribed=="YES" and $testos and $testbuild) { $approval_result = process_approval($type, $file, "approve"); + } else { + echo"Error: Approval cannot be processed because of missing data. Fill in the required fields and try again.
\n"; } } else { if ($testos and $testbuild and $comments) { $approval_result = process_approval($type, $file, "deny"); + } else { + echo"Error: Denial cannot be processed because of missing data. Fill in the required fields and try again.
\n"; } } } @@ -175,21 +183,21 @@ WHERE `approved` = '?' GROUP BY TV.URI ORDER BY TV.DateUpdated ASC"; echo"\n"; echo"\n"; echo"\n"; - echo"OSes: \n"; - echo"Apps: \n"; - echo"Comments: "; + echo"OSes: \n"; + echo"Apps: \n"; + echo"Comments (to author): "; echo"\n"; echo"\n"; - echo"Install?\n"; - echo"Uninstall?\n"; - echo"App Works? \n"; - echo"Clean Profile?\n"; + echo"Install?\n"; + echo"Uninstall?\n"; + echo"App Works? \n"; + echo"Clean Profile?\n"; if ($type=="E") { - echo"New Chrome?\n"; - echo"Works?\n"; + echo"New Chrome?\n"; + echo"Works?\n"; } else if ($type=="T") { - echo"Visual Errors?\n"; - echo"Theme Complete?\n"; + echo"Visual Errors?\n"; + echo"Theme Complete?\n"; } echo"\n"; diff --git a/mozilla/webtools/update/developers/inc_sidebar.php b/mozilla/webtools/update/developers/inc_sidebar.php index 602c8eb48d7..0d19b97928e 100755 --- a/mozilla/webtools/update/developers/inc_sidebar.php +++ b/mozilla/webtools/update/developers/inc_sidebar.php @@ -1,5 +1,5 @@