From d77cc42a2fd0ae059128043822ef127199e55ed2 Mon Sep 17 00:00:00 2001
From: Sebastian Riedel <sri@labs.kraih.com>
Date: Wed, 15 Sep 2004 21:27:52 +0000
Subject: [PATCH] XHTML update

git-svn-id: http://svn.maypole.perl.org/Maypole/trunk@179 48953598-375a-da11-a14b-00016c27c3ee
---
 templates/beer/addnew                 | 59 ++++++++++++++-------------
 templates/custom/frontpage            | 16 --------
 templates/custom/header               | 23 -----------
 templates/factory/addnew              |  8 ++--
 templates/factory/edit                |  4 +-
 templates/factory/frontpage           |  2 +-
 templates/factory/header              |  5 ++-
 templates/factory/list                |  6 +--
 templates/factory/macros              |  7 ++--
 templates/factory/search_form         | 10 ++---
 templates/factory/view                |  6 +--
 templates/{beerdb.css => maypole.css} |  0
 12 files changed, 55 insertions(+), 91 deletions(-)
 delete mode 100644 templates/custom/frontpage
 delete mode 100644 templates/custom/header
 rename templates/{beerdb.css => maypole.css} (100%)

diff --git a/templates/beer/addnew b/templates/beer/addnew
index e19c7c2..248fe28 100644
--- a/templates/beer/addnew
+++ b/templates/beer/addnew
@@ -1,31 +1,32 @@
 <h3>Add a new [%classmetadata.moniker%]</h3>
 [% USE element_maker = Class("HTML::Element") %]
-<P>
-<FORM METHOD="post" ACTION="[%base%]/[%classmetadata.moniker%]/do_edit/">
-    <INPUT TYPE="hidden" NAME="action" VALUE="create">
-    <INPUT TYPE="hidden" NAME="class" VALUE="[%classmetadata.name%]">
-    <TABLE class="view">
-     [% FOR col = classmetadata.columns;
-        NEXT IF col == "id";
-        SET element = classmetadata.cgi.$col;
-        IF col == "brewery";
-            SET create_url = base _ "/brewery/edit";
-            SET new_element = element_maker.new("option", 
-                "class", "redirect",
-                "value", create_url).push_content("Create new brewery");
-            SET child = element.content_list.first;
-            SET foo = element.attr("name", "navi");
-            SET child = child.attr("selected", "selected");
-            SET element =
-            element.unshift_content(element_maker.new("option", value," "));
-            element.attr("onChange", "go()");
-            SET element = element.unshift_content(new_element);
-        END;
-        %]
-     <TR> <TD class="field"> [% classmetadata.colnames.$col; %]
-     </TD>
-          <TD> [% element.as_HTML; %] </TD> </TR>
-    [% END; %]
-    </TABLE>
-    <INPUT TYPE="submit" NAME="create" VALUE="create">
-</FORM>
+<p>
+<form method="post" action="[% base %]/[% classmetadata.moniker %]/do_edit/">
+    <input type="hidden" name="action" value="create"/>
+    <input type="hidden" name="class" value="[% classmetadata.name %]"/>
+    <table class="view">
+        [% FOR col = classmetadata.columns;
+            NEXT IF col == "id";
+            SET element = classmetadata.cgi.$col;
+            IF col == "brewery";
+                SET create_url = base _ "/brewery/edit";
+                SET new_element = element_maker.new("option", 
+                    "class", "redirect",
+                    "value", create_url).push_content("Create new brewery");
+                SET child = element.content_list.first;
+                SET foo = element.attr("name", "navi");
+                SET child = child.attr("selected", "selected");
+                SET element =
+                element.unshift_content(element_maker.new("option", value," "));
+                element.attr("onChange", "go()");
+                SET element = element.unshift_content(new_element);
+            END;
+            %]
+            <tr>
+                <td class="field">[% classmetadata.colnames.$col; %]</td>
+                <td>[% element.as_HTML; %]</td>
+            </tr>
+        [% END; %]
+    </table>
+    <input type="submit" name="create" value="create"/>
+</form>
diff --git a/templates/custom/frontpage b/templates/custom/frontpage
deleted file mode 100644
index 2cef7ae..0000000
--- a/templates/custom/frontpage
+++ /dev/null
@@ -1,16 +0,0 @@
-[% INCLUDE header %]
-
-<h2> The beer database </h2>
-
-<TABLE BORDER="0" ALIGN="center" WIDTH="70%">
-[% FOR table = config.display_tables %]
-<TR>
-<TD>
-<A HREF="[%table%]/list">List by [%table %]</A>
-</TD>
-</TR>
-[% END %]
-</TABLE>
-
-<BR>
-[% INCLUDE footer %]
diff --git a/templates/custom/header b/templates/custom/header
deleted file mode 100644
index b95e32a..0000000
--- a/templates/custom/header
+++ /dev/null
@@ -1,23 +0,0 @@
-<HTML>
-    <HEAD>
-        <TITLE> Beer Database </TITLE>
-<META http-equiv=Content-Type content="text/html; charset=utf-8">
-
-<LINK title=myStyle href="/beerdb.css" type=text/css rel=stylesheet>
-
-<script language="javascript" type="text/javascript">
-<!--
-function go()
-{
-    box = document.forms[-1].navi;
-    destination = box.options[box.selectedIndex].value;
-    if (box.options[box.selectedIndex].class == 'redirect')
-        location.href = destination;
-}
-
-// -->
-</SCRIPT>
-</HEAD>
-<BODY>
-<DIV class="content">
-
diff --git a/templates/factory/addnew b/templates/factory/addnew
index f56fcf4..9345c78 100644
--- a/templates/factory/addnew
+++ b/templates/factory/addnew
@@ -11,9 +11,9 @@ table.
 
 #%]
 <h3>Add a new [% classmetadata.moniker %]</h3>
-<form method=post action="[% base %]/[% classmetadata.table %]/do_edit/">
-    <input type=hidden name=action value=create />
-    <table class=view>
+<form method="post" action="[% base %]/[% classmetadata.table %]/do_edit/">
+    <input type="hidden" name="action" value="create"/>
+    <table class="view">
     [% FOR col = classmetadata.columns %]
         [% NEXT IF col == "id" %]
         <tr>
@@ -22,5 +22,5 @@ table.
         </tr>
     [% END; %]
     </table>
-    <input type=submit name=create value=create />
+    <input type="submit" name="create" value="create"/>
 </form>
diff --git a/templates/factory/edit b/templates/factory/edit
index f1e0fbf..c5c511b 100644
--- a/templates/factory/edit
+++ b/templates/factory/edit
@@ -12,7 +12,7 @@ form similar to L<add_new> but with the current values filled in.
 [% INCLUDE header %]
 [% FOR item = objects; %]
 <h2>Edit [% item.name %]</h2>
-<form action="[% base %]/[% item.table %]/do_edit/[% item.id %]" method=post>
+<form action="[% base %]/[% item.table %]/do_edit/[% item.id %]" method="post">
 [% FOR col = classmetadata.columns;
     NEXT IF col == "id";
     "<p>";
@@ -23,6 +23,6 @@ form similar to L<add_new> but with the current values filled in.
     IF errors.$col; 
         errors.$col;
     END;
-    '<input type=submit name=edit value=edit />';
+    '<input type="submit" name="edit" value="edit"/>';
 END %]
 [% INCLUDE footer %]
diff --git a/templates/factory/frontpage b/templates/factory/frontpage
index 9cf3562..30861d2 100644
--- a/templates/factory/frontpage
+++ b/templates/factory/frontpage
@@ -12,7 +12,7 @@ It shows a list of all tables it is allowed to display.
 <h2>
     [% config.application_name || "A poorly configured Maypole application" %]
 </h2>
-<table border=0 align=center width=70%>
+<table border="0" align="center" width="70%">
 [% FOR table = config.display_tables %]
     <tr>
         <td>
diff --git a/templates/factory/header b/templates/factory/header
index 7e16e26..5a58823 100644
--- a/templates/factory/header
+++ b/templates/factory/header
@@ -8,8 +8,9 @@
                 "A poorly configured Maypole application"
             %]
         </title>
-        <meta http-equiv=Content-Type content="text/html; charset=utf-8" />
-        <link title=Maypole href=/maypole.css type=text/css rel=stylesheet />
+        <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
+        <link title="Maypole" href="/maypole.css" type="text/css"
+            rel="stylesheet"/>
         <script language="javascript" type="text/javascript">
             <!--
                 function go()
diff --git a/templates/factory/list b/templates/factory/list
index 8cd0209..120abbb 100644
--- a/templates/factory/list
+++ b/templates/factory/list
@@ -6,9 +6,9 @@
     <h2>Listing of all [% classmetadata.plural %]</h2>
 [% END %]
 [% INCLUDE navbar %]
-<div class=search>[% INCLUDE search_form %]</div>
-<div class=list>
-    <table id=matrix width=100%>
+<div class="search">[% INCLUDE search_form %]</div>
+<div class="list">
+    <table id="matrix" width="100%">
         <tr>
             [% FOR col = classmetadata.columns.list;
                 NEXT IF col == "id";
diff --git a/templates/factory/macros b/templates/factory/macros
index 2442dd5..3d45a6f 100644
--- a/templates/factory/macros
+++ b/templates/factory/macros
@@ -56,7 +56,7 @@ for some.
         NEXT IF col == "id";
         "<td>";
         IF col == "url";
-            "<A HREF="; item.url; "> "; item.url; "</A>";
+            "<a href="; item.url; "> "; item.url; "</a>";
         ELSIF col == item.stringify_column;
             maybe_link_view(item);
         ELSE;
@@ -78,8 +78,9 @@ This is a generic button, which performs an action on an object.
 #%]
 [% MACRO button(obj, action) BLOCK; %]
 <td>
-<form method=post action="[% base %]/[% obj.table %]/[% action %]/[% obj.id %]">
-    <input type=submit name=[% action %] value=[% action %] />
+<form method="post"
+    action="[% base %]/[% obj.table %]/[% action %]/[% obj.id %]">
+    <input type="submit" name="[% action %]" value="[% action %]"/>
 </form>
 </td>
 [% END %]
diff --git a/templates/factory/search_form b/templates/factory/search_form
index e5458f8..907ef7e 100644
--- a/templates/factory/search_form
+++ b/templates/factory/search_form
@@ -1,8 +1,8 @@
 <h3>Search</h3>
-<form method=get action="[% base %]/[% classmetadata.moniker %]/search/">
-    <input type=hidden name=action value=create />
-    <input type=hidden name=class value=[% classmetadata.name %] />
-    <table class=view>
+<form method="get" action="[% base %]/[% classmetadata.moniker %]/search/">
+    <input type="hidden" name="action" value="create"/>
+    <input type="hidden" name="class" value="[% classmetadata.name %]"/>
+    <table class="view">
         [% FOR col = classmetadata.columns;
             NEXT IF col == "id";
          %]
@@ -20,5 +20,5 @@
             </tr>
         [% END; %]
     </table>
-    <input type=submit name=search value=search>
+    <input type="submit" name="search" value="search"/>
 </form>
diff --git a/templates/factory/view b/templates/factory/view
index c00046c..6736da6 100644
--- a/templates/factory/view
+++ b/templates/factory/view
@@ -14,7 +14,7 @@ C<objects> and displays the object's properties in a table.
     [% SET string = item.stringify_column %]
     <h2> [% item.$string %]</h2>
     [% INCLUDE navbar %]
-    <table class=view>
+    <table class="view">
         <tr>
             <td class="field">[% classmetadata.colnames.$string %]</td>
             <td>[% item.$string %]</td>
@@ -32,10 +32,10 @@ from the C<column_names> method:
 
 #%]
             <tr>
-                <td class=field>[% classmetadata.colnames.$col; %]</td>
+                <td class="field">[% classmetadata.colnames.$col; %]</td>
                 <td>
                     [% IF col == "url";  # Possibly too much magic.
-                        "<A HREF="; item.url; "> "; item.url; "</A>";
+                        "<a href="; item.url; "> "; item.url; "</a>";
                     ELSE;
                         maybe_link_view(item.$col); 
                     END; %]
diff --git a/templates/beerdb.css b/templates/maypole.css
similarity index 100%
rename from templates/beerdb.css
rename to templates/maypole.css
-- 
2.39.2