Bug 22744: Remove the 'do not notify' buttons
authorNick Clemens <nick@bywatersolutions.com>
Thu, 13 Jun 2019 10:56:34 +0000 (10:56 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 23 Sep 2019 10:57:40 +0000 (11:57 +0100)
These checkboxes are simply a way to clear the prefs before editing. We are not saving
a lot of click, and we are confusing people, this patch simply removes them

To test:
1 - Edit a patron's messaging preferences
2 - Think too hard about the 'Do not notify' boxes
3 - Feel confused
4 - Apply this patch
5 - Edit a patron's messaging preferences
6 - Confirm it feels better

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc

index e0c880f..2f948c6 100644 (file)
@@ -11,7 +11,6 @@
         <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="fa fa-info-circle"></i></th>
         <!-- <th>RSS</th> -->
-        [% UNLESS ( messaging_form_inactive ) %]<th>Do not notify</th>[% END %]
     </tr>
     [% FOREACH messaging_preference IN messaging_preferences %]
     <tr>
       <td>-</td>
       [% END %] -->
 
-      [% UNLESS ( messaging_form_inactive ) %]
-      <td><input type="checkbox" class="none" id="none[% messaging_preference.message_attribute_id | html %]" /></td>
-      [% END %]
     </tr>
     [% END %]
   </table>