From 1bc4836e19223cd2bcdc46c517d16f80783c1bcf Mon Sep 17 00:00:00 2001
From: Loui Chang <louipc.ist@gmail.com>
Date: Mon, 23 Nov 2009 23:24:56 -0500
Subject: Restyle the layout.

Make HTML markup more logical.
Remove some unused style sheets rules.

Signed-off-by: Loui Chang <louipc.ist@gmail.com>
---
 web/html/css/arch.css  | 176 +++++++++++++++++--------------------------------
 web/html/css/fonts.css |   4 +-
 web/html/packages.php  |  10 +--
 web/html/pkgsubmit.php |   5 ++
 4 files changed, 73 insertions(+), 122 deletions(-)

(limited to 'web/html')

diff --git a/web/html/css/arch.css b/web/html/css/arch.css
index 6e2cd4a8..6bf4e119 100644
--- a/web/html/css/arch.css
+++ b/web/html/css/arch.css
@@ -2,142 +2,119 @@
 * ul { padding: 20px; }
 body {
   min-width: 600px;
-  background: #fbf8f1 url("../images/titlebar.png") repeat-x top left;
-  padding: 0 30px;
   color: #555;
-  font-family: sans-serif;
+  font-family: "DejaVu Sans", sans-serif;
+  font-size: 14px;
 }
 /* Divs */
-#head_container {
-  height: 220px;
+#title {
+  display: block;
+  position: relative;
+  height: 123px;
+}
+#main_nav,
+#sub_nav {
+  list-style: none;
+  font-size: 13px;
+  line-height: 26px;
+  padding: 0 32px;
 }
 #main_nav {
-  position: absolute;
-  top: 123px;
-  right: 30px;
-  width: 600px;
   background-color: #333;
+  text-align: right;
 }
 #sub_nav {
-  position: absolute;
-  top: 149px;
-  right: 30px;
   min-width: 700px;
   background-color: #1793d1;
+  text-align: right;
 }
-#main_nav ul,
-#sub_nav ul {
-  list-style: none;
-  margin: 0;
-  padding: 0;
-}
-#main_nav ul li,
-#sub_nav ul li {
-  display: block;
-  float: right;
-  margin: 0 3px;
-  padding: 0;
-  text-align: center;
-  font-size: 13px;
-  line-height: 26px;
-}
-#main_nav ul li.selected {
+#main_nav a.selected {
   background-color: #1793d1;
 }
-#sub_nav ul li.selected {
+#sub_nav a.selected {
   background-color: #333;
 }
-#main_nav ul li a,
-#sub_nav ul li a {
-  display: block;
-  padding: 0 8px;
+#main_nav a,
+#sub_nav a {
+  padding: 5px 9px;
+  line-height: 26px;
   color: #fff;
   font-weight: bold;
   text-decoration: none;
 }
-#main_nav ul li a:hover {
+#main_nav a:hover {
   background-color: #1793d1;
 }
-#lang_login_sub {
-  display: block;
-  position: relative;
-  top: -30px;
-  margin: auto;
-  width: 90%;
-}
-#lang_login_sub .button {
-  font-size: 9px;
-  margin: 2px;
-}
-#login_bar {
-  font-size: 10px;
-  display: block;
-  float: left;
-}
-#login_bar input {
-  margin-right: 5px;
+#sub_nav a:hover {
+  background-color: #333;
 }
-#lang_bar { 
+#lang_sub {
   display: block;
+  font-size: 10pt;
+  text-align: right;
   position: relative;
-  float: right;
+  margin-bottom: 10px;
+  margin-right: 35px;
 }
-#lang_login_sub ul {
+#lang_sub ul {
   list-style: none;
-  padding: 0;
+  padding: 0px;
 }
-#lang_login_sub ul a {
+#lang_sub ul a {
   text-decoration: none;
 }
-#lang_login_sub ul a:hover {
+#lang_sub ul a:hover {
   text-decoration: underline;
 }
-#lang_login_sub ul li {
+#lang_sub ul li {
   display: inline;
   padding-right: 1px;
-  font-size: 10px;
+}
+#login_bar {
+	display: block;
+	margin-bottom: 40px;
+}
+#login_bar input {
+  margin-right: 5px;
+}
+#login_bar .button {
+  margin: 2px;
 }
 #updates {
   font-size: small;
   /*position: relative;*/
-  top: 0;
+  top: 0px;
   background-color: #e7f0f6;
   border: 1px solid #92cbe8;
   padding: 10px;
 }
-#ads {
-  z-index: 0;
-  position: absolute;
-  top: 40px;
-  right: 10px;
-}
 #logo {
   z-index: 1;
-  position: absolute;
+  position: relative;
   top: 10px;
   left: 25px;
   width: 350px;
 }
 #logo h1#archtitle {
-  text-indent: -9999px;
-  margin: 0;
-  padding: 0;
+  background: transparent url("../images/titlelogo.png") no-repeat top left;
+  margin: 0px;
+  padding: 0px;
 }
 #logo h1#archtitle a {
+  color: transparent;
   display: block;
-  background: transparent url("../images/titlelogo.png") top left no-repeat;
   width: 350px;
   height: 103px;
 }
 .clear {
   clear: both;
-  margin: 0;
-  padding: 0;
+  margin: 0px;
+  padding: 0px;
 }
 .right {
   float: right;
   width: 320px;
-  padding: 0 10px 10px 0;
+  padding: 0px 10px 10px 0px;
 }
 .left {
   padding: 10px;
@@ -182,7 +159,7 @@ div.listing {
 }
 /* Headers */
 h2 {
-  margin: 20px 0 10px 0;
+  margin: 20px 0px 10px 0px;
 }
 h2.title {
   border-bottom: 1px solid #46494d;
@@ -235,7 +212,7 @@ ul.small {
 ul.links {
   list-style: none;
   font-size: small;
-  padding: 0 0 20px 20px;
+  padding: 0px 0px 20px 20px;
 }
 img {
   border: none;
@@ -265,30 +242,8 @@ table.center {
   margin-left: auto;
   margin-right: auto;
 }
-table#releases {
-  font-size: small;
-  width: 100%;
-}
-table#releases td {
-  padding-right: 20px;
-}
-table#repolinks {
-  font-size: small;
-  width: 100%;
-}
-table#repolinks td {
-  text-align: right;
-}
-table#repolinks th {
-  text-align: left;
-}
-table#art {
-  text-align: center;
-  margin-left: auto;
-  margin-right: auto;
-}
 table.results {
-  padding: 0;
+  padding: 0px;
   border-collapse: collapse;
 }
 .results th {
@@ -296,7 +251,7 @@ table.results {
   border-bottom: 1px solid #46494d;
   border-top: 1px solid #46494d;
   text-align: left;
-  padding: 0 5px 0 0;
+  padding: 0px 5px 0px 0px;
 }
 .results th>a {
   text-decoration: none;
@@ -329,20 +284,9 @@ blockquote.code {
   font-size: .95em;
 }
 
-/* Used by Django's FormWrappers */
-textarea.vLargeTextField {
-  width: 450px;
-  height: 250px;
-}
-.pkgr2 {
-  background-color: #eee4cb;
-}
-#maincontent {
-  width: 100%;
-  text-align: left;
-  clear: both;
-}
+/* AUR Styles */
 .pgbox {
+  margin: 0 25px;
   margin-bottom: 1%;
   background-color: #fff;
   border: 2px solid #ddd;
@@ -372,7 +316,7 @@ textarea.vLargeTextField {
 }
 .pgboxbody-comment > table {
   border-collapse: separate;
-  border-spacing: 0 15px;
+  border-spacing: 0px 15px;
 }
 
 #advanced-search li {
diff --git a/web/html/css/fonts.css b/web/html/css/fonts.css
index de70d76d..beaf4e22 100644
--- a/web/html/css/fonts.css
+++ b/web/html/css/fonts.css
@@ -105,7 +105,9 @@ h1, h2, h3, h4 {
 	color: #f00;
 }
 .pkgoutput {
+	font-size: 12pt;
 	font-weight: bold;
-	text-align: center;
+	text-align: left;
+	clear: both;
 }
 
diff --git a/web/html/packages.php b/web/html/packages.php
index ace45995..b2c3f627 100644
--- a/web/html/packages.php
+++ b/web/html/packages.php
@@ -55,13 +55,13 @@ if ($_POST['action'] == "do_Flag" || isset($_POST['do_Flag'])) {
 }
 
 html_header($title);
-
-if ($output):
 ?>
-<div class="pkgoutput"><?php print $output ?></div><br />
-<?php
-endif;
 
+<?php if ($output): ?>
+	<p class="pkgoutput"><?php print $output ?></p>
+<?php endif; ?>
+
+<?php
 if (isset($_GET['ID'])) {
 	include('pkg_search_form.php');
 	if (!$_GET['ID'] = intval($_GET['ID'])) {
diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php
index dfe3ce8d..ac98ba75 100644
--- a/web/html/pkgsubmit.php
+++ b/web/html/pkgsubmit.php
@@ -430,6 +430,10 @@ html_header("Submit");
 
 ?>
 
+<?php if ($error): ?>
+	<p class="pkgoutput"><?php print $error ?></p>
+<?php endif; ?>
+
 <div class="pgbox">
 	<div class="pgboxtitle">
 		<span class="f3"><?php print __("Submit"); ?></span>
@@ -502,6 +506,7 @@ html_header("Submit");
 else:
 	# Visitor is not logged in
 	print __("You must create an account before you can upload packages.");
+	exit();
 ?>
 
 <br />
-- 
cgit v1.2.3-24-g4f1b