From 6babeeccabd71c0cd7990847d5c615012740ffdb Mon Sep 17 00:00:00 2001
From: Aleisha Amohia <aleishaamohia@hotmail.com>
Date: Tue, 27 Jun 2017 01:59:55 +0000
Subject: [PATCH] Bug 18860: Adding thead tags to OPAC messaging settings table

So that it can be styled alongside other tables in the OPAC.

To test:
1) Log into staff side
2) Find OPACUserCSS syspref and add the following CSS:
thead {
color: red;
}
3) Log into the OPAC
4) Go to your fines, your search history, your reading history etc -
notice all of these table's headings are styled with red text
5) Go to your messaging. Notice this table's headings are not styled
with red text
6) Apply patch and refresh page
7) Your messaging table should now have red headings.
(You can delete the CSS from OPACUserCSS if you'd like.)

Sponsored-by: Catalyst IT

Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 .../bootstrap/en/modules/opac-messaging.tt         | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
index 84cb228130..ed7d468db2 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
@@ -32,15 +32,18 @@
                         <input type="hidden" name="modify" value="yes" />
 
                             <table class="table table-bordered table-condensed table-striped">
-                                <tr>
-                                    <th>&nbsp;</th>
-                                    <th>Days in advance</th>
-                                    [% IF ( SMSSendDriver ) %]<th>SMS</th>[% END %]
-                                    [% IF ( TalkingTechItivaPhone ) %]<th>Phone</th>[% END %]
-                                    <th>Email</th>
-                                    <th>Digests only <i id="info_digests" data-toggle="tooltip" title="You can ask for a digest to reduce the number of messages. Messages will be saved and sent as a single message." data-placement="right" class="icon icon-info-sign"></i></th>
-                                    <!-- <th>RSS</th> --><th>Do not notify</th>
-                                </tr>
+                                <thead>
+                                    <tr>
+                                        <th>&nbsp;</th>
+                                        <th>Days in advance</th>
+                                        [% IF ( SMSSendDriver ) %]<th>SMS</th>[% END %]
+                                        [% IF ( TalkingTechItivaPhone ) %]<th>Phone</th>[% END %]
+                                        <th>Email</th>
+                                        <th>Digests only <i id="info_digests" data-toggle="tooltip" title="You can ask for a digest to reduce the number of messages. Messages will be saved and sent as a single message." data-placement="right" class="icon icon-info-sign"></i></th>
+                                        <!-- <th>RSS</th> --><th>Do not notify</th>
+                                    </tr>
+                                </thead>
+                                <tbody>
                                 [% FOREACH messaging_preference IN messaging_preferences %]
                                     <tr>
                                         <td>[% IF ( messaging_preference.Item_Due ) %]Item due
@@ -117,6 +120,7 @@
                                         <td><input type="checkbox" class="none" id="none[% messaging_preference.message_attribute_id %]" /></td>
                                     </tr>
                                 [% END # / FOREACH messaging_preferences%]
+                            </tbody>
                         </table>
 
                         <fieldset class="rows">
-- 
2.11.0