From 68229edf83a7a56db56facb6ca7a3f26627f070a Mon Sep 17 00:00:00 2001
From: Galen Charlton <gmc@esilibrary.com>
Date: Tue, 31 Mar 2015 20:34:42 +0000
Subject: [PATCH] LP#1442246: improve styling of patron messages

This patch adds new CSS classes for pieces of the
patron message display, uses a preformatted display
for the message body, and fixes an HTML nesting error.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
---
 Open-ILS/src/templates/opac/css/style.css.tt2          |  8 ++++++++
 .../templates/opac/myopac/messages/single_message.tt2  | 10 +++++-----
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 1b50d0f9c6..2c63646f37 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1735,6 +1735,14 @@ a.preflib_change {
    display:none;
 }
 
+/* patron message center */
+#myopac_message_tbody {
+    vertical-align: top;
+}
+.myopac_message_message {
+    white-space: pre;
+}
+
 @media only screen and (max-width: 800px) {
     .facet_sidebar_hidden, .result_block_hidden {
         display: none;
diff --git a/Open-ILS/src/templates/opac/myopac/messages/single_message.tt2 b/Open-ILS/src/templates/opac/myopac/messages/single_message.tt2
index 3d6a82c444..79ae7a12de 100644
--- a/Open-ILS/src/templates/opac/myopac/messages/single_message.tt2
+++ b/Open-ILS/src/templates/opac/myopac/messages/single_message.tt2
@@ -25,27 +25,27 @@
         <table title="[% l('Message') %]"
             class='light_border data_grid'>
             <tbody id='myopac_message_tbody'>
-            </tbody>
             <tr>
                 <td width='30%'
                     class='color_4 light_border'>[% l("Date") %]</td>
-                <td>[% date.format(ctx.parse_datetime(ctx.patron_messages.0.create_date), DATE_FORMAT); %]</td>
+                <td class='myopac_message_date'>[% date.format(ctx.parse_datetime(ctx.patron_messages.0.create_date), DATE_FORMAT); %]</td>
             </tr>
             <tr>
                 <td width='30%'
                     class='color_4 light_border'>[% l("Library") %]</td>
-                <td>[% ctx.patron_messages.0.library | html %]</td>
+                <td class='myopac_message_library'>[% ctx.patron_messages.0.library | html %]</td>
             </tr>
             <tr>
                 <td width='30%'
                     class='color_4 light_border'>[% l("Subject") %]</td>
-                <td>[% ctx.patron_messages.0.title | html %]</td>
+                <td class='myopac_message_subject'>[% ctx.patron_messages.0.title | html %]</td>
             </tr>
             <tr>
                 <td width='30%'
                     class='color_4 light_border'>[% l("Message") %]</td>
-                <td>[% ctx.patron_messages.0.message | html %]</td>
+                <td class='myopac_message_message'>[% ctx.patron_messages.0.message | html %]</td>
             </tr>
+            </tbody>
         </table>
     </div>
 </div>
-- 
2.43.2