From c7b9c42c6757552f65c78f040f05b99309270ee2 Mon Sep 17 00:00:00 2001
From: Thierry Pasqualier <thierry@uncensored.citadel.org>
Date: Sun, 20 May 2007 14:57:53 +0000
Subject: [PATCH] Summary Banner : HTML/CSS simplification

I would like to handle in the same way all the banners
of services and administration screens
at the end to reduce the number of classes CSS
---
 webcit/static/webcit.css | 21 +++++++++++++++++----
 webcit/summary.c         | 27 +++++++++++++--------------
 2 files changed, 30 insertions(+), 18 deletions(-)

diff --git a/webcit/static/webcit.css b/webcit/static/webcit.css
index f7dcfa7cb..184c50919 100644
--- a/webcit/static/webcit.css
+++ b/webcit/static/webcit.css
@@ -71,13 +71,12 @@ body {
 	background: #445;
 }
 
-
-#banner .room_banner {
+#banner .room_banner, #banner .service_banner {
         float: left;
         width: 48%;
 }
 
-.room_banner img {
+.room_banner img,  .service_banner img {
         float: left;
         margin-top: 0;
         margin-left: 0;
@@ -93,6 +92,20 @@ body {
         padding: 2px;
 }
 
+.service_banner h1 {
+        font-size: 12pt;
+        font-weight: bold;
+        color: #FFFFEE;
+        margin: 0;
+        padding: 2px;
+}
+
+.service_banner h2 {
+        font-size: 11pt;
+        font-weight: bold;
+        color: #FFFFEE;
+}
+
 .room_banner .infos {
         font-size: 8pt;
 	color: #FFFFEE;
@@ -845,7 +858,7 @@ div.auto_complete ul strong.highlight {
         background-color: #444455;
 }
 
-.smtpqueue_banner, .summary_banner, .useredit_banner, .userlist_banner, .downloads_banner, .vcard_edit_banner, .who_banner, .room_banner   {
+.smtpqueue_banner, .useredit_banner, .userlist_banner, .downloads_banner, .vcard_edit_banner, .who_banner, .room_banner   {
         border: 0;
         width: 100%;
         background-color: #444455;
diff --git a/webcit/summary.c b/webcit/summary.c
index 13dfdf70a..95fa8a8a4 100644
--- a/webcit/summary.c
+++ b/webcit/summary.c
@@ -251,26 +251,25 @@ void summary(void) {
 
 	output_headers(1, 1, 2, 0, 0, 0);
 	wprintf("<div id=\"banner\">\n");
-	wprintf("<table class=\"summary_banner\"><tr>"
-		"<td><img src=\"static/summscreen_48x.gif\"></td><td>"
-		"<span class=\"titlebar\">"
-	);
-
-	snprintf(title, sizeof title, _("Summary page for %s"), WC->wc_fullname);
-	escputs(title);
-	wprintf("</span></td><td>\n");
-	wprintf("</td><td aling=right><span class=\"titlebar\">");
-	output_date();
-	wprintf("</span><br />");
-	offer_start_page();
-	wprintf("</td></tr></table>\n");
+        wprintf("<div class=\"service_banner\">\n");
+        wprintf("<img src=\"static/summscreen_48x.gif\">");
+        wprintf("<h1>");
+        snprintf(title, sizeof title, _("Summary page for %s"), WC->wc_fullname);
+        escputs(title);
+        wprintf("</h1><h2>\n");
+        output_date();
+        wprintf("</h2></div>");
+        wprintf("<ul><li class=\"start_page\">");
+        offer_start_page();
+        wprintf("</li></ul>");
+        wprintf("</div>");
 
 	/**
 	 * You guessed it ... we're going to refresh using ajax.
 	 * In the future we might consider updating individual sections of the summary
 	 * instead of the whole thing.
 	 */
-	wprintf("</div>\n<div id=\"content\">\n");
+	wprintf("<div id=\"content\">\n");
 	summary_inner_div();
 	wprintf("</div>\n");
 
-- 
2.39.2