From 48b22259cab4c2108c6508c7886e285b596db67c Mon Sep 17 00:00:00 2001
From: Roland Haeder <roland@mxchange.org>
Date: Fri, 21 Aug 2015 11:32:12 +0200
Subject: [PATCH] =?utf8?q?Fixes=20for=20bad=20HTML=20with=20better=20HTML?=
 =?utf8?q?=20and=20CSS=20Signed-off-by:Roland=20H=C3=A4der=20<roland@mxcha?=
 =?utf8?q?nge.org>?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 web/index.jsp | 72 +++++++++++++++++++++++++--------------------------
 web/style.css |  8 ++++++
 2 files changed, 44 insertions(+), 36 deletions(-)

diff --git a/web/index.jsp b/web/index.jsp
index 69f8b768..916ccd50 100644
--- a/web/index.jsp
+++ b/web/index.jsp
@@ -41,59 +41,59 @@
 
 			<div id="content">
 				<table class="table">
-					<thead>
-						<tr>
-							<th class="table_header_column">
-								Bestellen?
-							</th>
-							<th class="table_header_column">
-								Anzahl:
-							</th>
-							<th class="table_header_column">
-								Produkt:
-							</th>
-							<th class="table_header_column">
-								Einzelpreis:
-							</th>
-						</tr>
-					</thead>
+					<tr>
+						<th class="table_header_column">
+							Bestellen?
+						</th>
+						<th class="table_header_column">
+							Anzahl:
+						</th>
+						<th class="table_header_column">
+							Produkt:
+						</th>
+						<th class="table_header_column">
+							Einzelpreis:
+						</th>
+					</tr>
+				</table>
 
-					<tbody>
-						<%
-						// Get Iterator
-						Iterator<Product> iterator = app.getAvailableProducts();
+				<%
+				// Get Iterator
+				Iterator<Product> iterator = app.getAvailableProducts();
 
-						// "Walk" through all products and unmark them as ordered
-						while (iterator.hasNext()) {
-							// Get product instance
-							Product product = iterator.next();
+				// "Walk" through all products and unmark them as ordered
+				while (iterator.hasNext()) {
+					// Get product instance
+					Product product = iterator.next();
 
-							// Unmark as ordered
-							app.unmarkProductAsOrdered(product, session);
-							%>
+					// Unmark as ordered
+					app.unmarkProductAsOrdered(product, session);
+					%>
+				<form action="<%=request.getContextPath()%>/form_handler/add_item.jsp" accept-charset="utf-8" id="form" method="post">
+				<table class="table">
+					<tbody>
 							<tr>
-								<form action="<%=request.getContextPath()%>/form_handler/add_item.jsp" accept-charset="utf-8" id="form" method="post">
-								<td>
+								<td class="table_data_column">
 									<input class="submit" type="submit" name="add" value="Hinzuf&uuml;gen" />
 									<input class="input" type="hidden" name="<%=PizzaApplication.HTTP_PARAM_ITEM_ID%>" value="<%=product.getId()%>" />
 									<input class="input" type="hidden" name="<%=PizzaApplication.HTTP_PARAM_ITEM_TYPE%>" value="<%=Product.class%>" />
 								</td>
-								<td>
+								<td class="table_data_column">
 									<input class="input" type="text" name="<%=PizzaApplication.HTTP_PARAM_AMOUNT%>" size="3" maxlength="20" />
 								</td>
-								<td>
+								<td class="table_data_column">
 									<%=product.getTitle()%>
 								</td>
-								<td align="right">
+								<td class="table_data_column" align="right">
 									<fmt:formatNumber type="currency"><%=product.getPrice()%></fmt:formatNumber>
 								</td>
 							</tr>
-							</form>
-							<%
-						}
-						%>
 					</tbody>
 				</table>
+				</form>
+				<%
+			}
+			%>
 			</div>
 		</div>
 
diff --git a/web/style.css b/web/style.css
index ca487498..b44a60ae 100644
--- a/web/style.css
+++ b/web/style.css
@@ -60,3 +60,11 @@ ul.footer_nav li {
 	list-style: none;
 	padding-left: 5px;
 }
+
+.table_header_column {
+	width: 100px;
+}
+
+.table_data_column {
+	width: 100px;
+}
-- 
2.39.5