This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Evergreen ILS". The branch, main has been updated via 7ddcbb35f0f373ad161282b40252e68f6b1ed046 (commit) from 34ea53321cbcc50dff3cdce0c0767a6f923d667b (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 7ddcbb35f0f373ad161282b40252e68f6b1ed046 Author: Terran McCanna <tmccanna@georgialibraries.org> Date: Wed Feb 5 13:39:21 2025 -0500 LP2090958 Can't select charges for credit card payment The OPAC accessibility omnibus included some changes that broke the functionality and layout of the My Account > Charges page. This partially reverts the changes to the charges.tt2 page in order to restore functionality and layout until the page can be reviewed again for accessibility needs. Release-note: Restores the ability to select charges for payment in the OPAC. Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org> Signed-off-by: Llewellyn Marshall <llewellyn.marshall@dncr.nc.gov> Signed-off-by: Michele Morgan <mmorgan@noblenet.org> diff --git a/Open-ILS/src/templates-bootstrap/opac/myopac/charges.tt2 b/Open-ILS/src/templates-bootstrap/opac/myopac/charges.tt2 index 7d3b06b4d3..612d4649f5 100755 --- a/Open-ILS/src/templates-bootstrap/opac/myopac/charges.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/myopac/charges.tt2 @@ -72,14 +72,14 @@ END %] <tr role="row" id='myopac_circ_trans_row'> [% IF myopac_cc_allowed %] - <th class="checkCell form-check"> + <td class="checkCell"> <input type="checkbox" checked="checked" - [% html_text_attr('aria-label', l('Pay [_1] fine', money(f.xact.balance_owed))) %] - name="xact" value="[% f.xact.id %]" class="form-check-input fineCheck"/> - </th> + [% html_text_attr('aria-label', l('Pay [_1] fine', money(f.xact.balance_owed))) %] + name="xact" value="[% f.xact.id %]" class="fineCheck"/> + </td> [% END %] - <td role="cell" class="numeric money fw-bold" mobile-title='[% l("Owed") %]'> + <td role="cell" class="numeric money fw-bold danger" mobile-title='[% l("Owed") %]'> [% money(f.xact.balance_owed) %] <input class="fineAmount" value="[% f.xact.balance_owed %]" hidden /> </td> @@ -151,7 +151,7 @@ [% IF myopac_cc_allowed %] <div class="d-block d-md-none form-check"> - <input id="pay_fines_check" checked="checked" class="form-check-input" + <input id="pay_fines_check" checked="checked" class="form-check-input" type="checkbox" onclick="select_all_checkboxes('xact_misc', this.checked)" aria-label="[% l('Check/Uncheck All') %]" /> <label for="pay_fines_check" class="form-check-label">[% l('Check/Uncheck All') %]</label> @@ -165,7 +165,7 @@ [% IF myopac_cc_allowed %] <th style="white-space:nowrap;"> - <input id="pay_fines_box2" checked="checked" class="form-check-input" + <input id="pay_fines_box2" checked="checked" type="checkbox" onclick="select_all_checkboxes('xact_misc', this.checked)" aria-label="[% l('Check/Uncheck All') %]" /> </th> @@ -180,28 +180,29 @@ </thead> <tbody id='myopac_trans_tbody'> [% FOR f IN ctx.fines.grocery %] - <tr role="row" class ='myopac_trans_row'> + <tr role="row" class='myopac_trans_row'> [% c = c + 1; %] [% IF myopac_cc_allowed %] - <th role="rowheader" scope="row" class="checkCell form-check"> + <td class="checkCell"> <label class="sr-only">[% l('Billing Number [_1]', c) %]</label> - <input aria-label="[% l('Pay [_0] bill', money(f.xact.balance_owed)) %]" class="form-check-input fineCheck" type="checkbox" name="xact_misc" value="[% f.xact.id %]" checked="checked" /> - </th> + <input aria-label="[% l('Pay [_0] bill', money(f.xact.balance_owed)) %]" class="fineCheck" + type="checkbox" name="xact_misc" value="[% f.xact.id %]" checked="checked" /> + </td> [% END %] <td role="cell" class="danger"> - <strong> - [% money(f.xact.balance_owed) %] - </strong> - <input type="hidden" class="fineAmount" value="[% f.xact.balance_owed %]" hidden /> + [% money(f.xact.balance_owed) %] + <input type="hidden" class="fineAmount" value="[% f.xact.balance_owed %]" hidden /> </td> <td role="cell"> + <span class="sr-only">[% l('Billing For') %]</span> [% f.xact.last_billing_type %] </td> <td role="cell"> [% IF f.xact.last_billing_note; %] + <span class="sr-only">[% l('Note') %]</span> [% f.xact.last_billing_note %] [% END %] </td> @@ -221,7 +222,7 @@ <div>[% l('You have no current fines.') %]</div> [% ELSIF myopac_cc_allowed %] <div class="my-3"> - <button type="button" value="[% l('Pay selected charges') %] - [% money(ctx.user_stats.fines.balance_owed) %]" title="[% l('Pay selected charges') %]" class="btn btn-confirm my-1" + <button type="submit" value="[% l('Pay selected charges') %] - [% money(ctx.user_stats.fines.balance_owed) %]" title="[% l('Pay selected charges') %]" class="btn btn-confirm my-1" id="selectPay"> <i class='fas fa-money-bill-wave' aria-hidden="true"></i> [% l('Pay Selected Charges') %] - [% money(ctx.user_stats.fines.balance_owed) %]</button> <button type="submit" class="btn btn-confirm my-1"><i class="fas fa-cash-register" aria-hidden="true"></i> [% l('Pay All Charges') %] - [% money(ctx.user_stats.fines.balance_owed) %]</button> </div> ----------------------------------------------------------------------- Summary of changes: .../templates-bootstrap/opac/myopac/charges.tt2 | 33 +++++++++++----------- 1 file changed, 17 insertions(+), 16 deletions(-) hooks/post-receive -- Evergreen ILS