UI work

git-svn-id: http://svn.igniterealtime.org/svn/repos/messenger/trunk@442 b35dd754-fafc-0310-a699-88a17e54d16e

src/web/muc-history-settings.jsp

@@ -21,20 +21,6 @@
 <%@ taglib uri="core" prefix="c"%>
 <%@ taglib uri="fmt" prefix="fmt"%>
 
-<%-- Define Administration Bean --%>
-<jsp:useBean id="admin" class="org.jivesoftware.util.WebManager"/>
-<%  admin.init(request, response, session, application, out ); %>
-<jsp:useBean id="pageinfo" scope="request" class="org.jivesoftware.admin.AdminPageBean" />
-<%  // Title of this page and breadcrumbs
-    String title = "Group Chat History Settings";
-    pageinfo.setTitle(title);
-    pageinfo.getBreadcrumbs().add(new AdminPageBean.Breadcrumb("Main", "index.jsp"));
-    pageinfo.getBreadcrumbs().add(new AdminPageBean.Breadcrumb("Group Chat History", "muc-history-settings.jsp"));
-    pageinfo.setPageID("muc-history");
-%>
-<jsp:include page="top.jsp" flush="true" />
-<jsp:include page="title.jsp" flush="true" />
-
 <%!  // Global vars and methods:
 
     // Strategy definitions:
@@ -42,6 +28,10 @@
     static final int NONE = 2;
     static final int NUMBER = 3;
 %>
+
+<jsp:useBean id="admin" class="org.jivesoftware.util.WebManager"/>
+<%  admin.init(request, response, session, application, out ); %>
+
 <%   // Get parameters:
     boolean update = request.getParameter("update") != null;
     int policy = ParamUtils.getIntParameter(request,"policy",-1);
@@ -76,9 +66,8 @@
                 historyStrat.setMaxNumber(numMessages);
             }
             // All done, redirect
-            %>
-            <p class="jive-success-text">Settings updated.</p>
-            <% 
+            response.sendRedirect("muc-history-settings.jsp?success=true");
+            return;
         }
     }
 
@@ -97,14 +86,45 @@
     }
 %>
 
-  <table cellpadding="3" cellspacing="1" border="0" width="600">
-<form action="muc-history-settings.jsp">
+<jsp:useBean id="pageinfo" scope="request" class="org.jivesoftware.admin.AdminPageBean" />
+<%  // Title of this page and breadcrumbs
+    String title = "Group Chat History Settings";
+    pageinfo.setTitle(title);
+    pageinfo.getBreadcrumbs().add(new AdminPageBean.Breadcrumb("Main", "index.jsp"));
+    pageinfo.getBreadcrumbs().add(new AdminPageBean.Breadcrumb("Group Chat History", "muc-history-settings.jsp"));
+    pageinfo.setPageID("muc-history");
+%>
+<jsp:include page="top.jsp" flush="true" />
+<jsp:include page="title.jsp" flush="true" />
+
+<p>
+Group Chat rooms can replay conversation histories to provide context to new members joining a room.
+<fmt:message key="short.title" bundle="${lang}"/> provides several options for controlling how much
+history to store for each room.
+</p>
 
-    <tr>
-      <td class="text" colspan="2">Group Chat rooms can replay conversation histories to provide context to new members joining a room.
-        <fmt:message key="short.title" bundle="${lang}"/> provides several options for controlling how much history to store for each room.
-      </td>
-    </tr>
+<%  if ("true".equals(request.getParameter("success"))) { %>
+
+    <div class="jive-success">
+    <table cellpadding="0" cellspacing="0" border="0">
+    <tbody>
+        <tr><td class="jive-icon"><img src="images/success-16x16.gif" width="16" height="16" border="0"></td>
+        <td class="jive-icon-label">
+        Settings updated successfully.
+        </td></tr>
+    </tbody>
+    </table>
+    </div><br>
+
+<%  } %>
+
+<form action="muc-history-settings.jsp" method="post">
+
+<fieldset>
+    <legend>Set Conflict Policy</legend>
+    <div>
+    <table cellpadding="3" cellspacing="0" border="0" width="100%">
+    <tbody>
         <tr valign="top" class="">
             <td width="1%" nowrap>
                 <input type="radio" name="policy" value="<%= NONE %>" id="rb01"  <%= ((policy==NONE) ? "checked" : "") %> />
@@ -141,9 +161,15 @@
                 <input type="text" name="numMessages" size="5" maxlength="10" onclick="this.form.policy[2].checked=true;" value="<%= ((numMessages > 0) ? ""+numMessages : "") %>"/> messages
             </td>
         </tr>
+    </tbody>
     </table>
-  <br/>
-  <input type="submit" name="update" value="Save Settings"/>
+    </div>
+</fieldset>
+
+<br><br>
+
+<input type="submit" name="update" value="Save Settings"/>
+
 </form>
-<%@ include file="bottom.jsp"%>
 
+<jsp:include page="bottom.jsp" flush="true" />
\ No newline at end of file