Bug 22744: Remove the 'do not notify' buttons
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>
(cherry picked from commit ff21f97908)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
parent
f05a3767f6
commit
02cc140d68
|
|
@ -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>
|
||||
|
|
@ -190,9 +189,6 @@
|
|||
<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>
|
||||
|
|
|
|||
Loading…
Reference in New Issue