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 a4bf46d1bd513ca6f30389dd6e53093fcc9e9f9d (commit) from 4224db38675a9ab80471af2c67c4f9bdad641c50 (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 a4bf46d1bd513ca6f30389dd6e53093fcc9e9f9d Author: Stephanie Leary <stephanie.leary@equinoxoli.org> Date: Thu Apr 17 16:22:05 2025 +0000 LP2107576 "Show more copies" should skip to row ID In the OPAC record detail page, the table rows should have IDs to serve as URL fragments, and the "Show more copies" link should lead to the row showing the first additional copy, skipping the user past the rows they have already seen. Release-note: OPAC "Show more copies" link now leads directly to the URL fragment of the first additional copy Sponsored-by: BC Libraries Cooperative Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org> Signed-off-by: Ruth Frasur Davis <rfrasur@gmail.com> Signed-off-by: IanSkelskey <ianskelskey@gmail.com> Signed-off-by: Jane Sandberg <js7389@princeton.edu> diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_table.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_table.tt2 index 7b4e132ad1..7ae9349a05 100755 --- a/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_table.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_table.tt2 @@ -155,7 +155,7 @@ END; # FOREACH bib callnum = callnum _ " " _ callnum_suffix; END; -%] - <tr role="row" property="offers" typeof="Offer" mobile-count="#[% loop.count %]"> + <tr role="row" id="row[% loop.count %]" property="offers" typeof="Offer" mobile-count="#[% loop.count %]"> [%- IF serial_holdings %] <td role="cell" class='rdetail-issue-issue'> [%- copy_info.holding_label | html; -%] @@ -377,7 +377,7 @@ END; # FOREACH bib [%- more_copies_limit = 50 %] [%# TODO: config %] [%- IF ctx.copy_limit != more_copies_limit AND copies.size >= ctx.copy_limit AND ctx.copy_limit < ctx.total_copies %] <div class="rdetail_show_copies"> - <a href="[% mkurl('', {copy_limit => more_copies_limit, copy_offset => 0}) %]"><i class="fas fa-plus-square"></i> [% l('Show more copies') %]</a> + <a href="[% mkurl('', {copy_limit => more_copies_limit, copy_offset => 0}, '', 'row' _ (copies.size + 1)) %]"><i class="fas fa-plus-square"></i> [% l('Show more copies') %]</a> </div> [%- ELSIF ctx.copy_limit == more_copies_limit %] <div class="rdetail_show_copies"> diff --git a/Open-ILS/src/templates/kpac/parts/copy_table.tt2 b/Open-ILS/src/templates/kpac/parts/copy_table.tt2 index 7427220d29..50c118acbb 100644 --- a/Open-ILS/src/templates/kpac/parts/copy_table.tt2 +++ b/Open-ILS/src/templates/kpac/parts/copy_table.tt2 @@ -35,7 +35,7 @@ END; callnum = callnum _ " " _ callnum_suffix; END; -%] - <tr> + <tr id="row[% loop.count %]"> <td class="location"> [%- INCLUDE "opac/parts/library_name_link.tt2"; -%] </td> @@ -72,7 +72,7 @@ END; [%- more_copies_limit = 50 %] [%# TODO: config %] <td> [%- IF ctx.copy_limit != more_copies_limit AND ctx.copies.size >= ctx.copy_limit %] - <a href="[% mkurl('', {copy_limit => more_copies_limit, copy_offset => 0}) %]">[% l('Show more copies') %]</a> + <a href="[% mkurl('', {copy_limit => more_copies_limit, copy_offset => 0}, '', 'row' _ (copies.size + 1)) %]">[% l('Show more copies') %]</a> [%- ELSIF ctx.copy_limit == more_copies_limit %] <a href="[% mkurl('', {copy_limit => 0, copy_offset => 0}) %]">[% l('Show fewer copies') %]</a> [%- END %] diff --git a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 index 23988f8096..c071a7fca2 100644 --- a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 +++ b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 @@ -371,13 +371,13 @@ END; # FOREACH bib [%- END %] </tr> [% IF NOT serial_holdings -%] - <tr> + <tr id="row[% loop.count %]"> <td> [%- more_copies_limit = 50 %] [%# TODO: config %] [%- IF ctx.copy_limit != more_copies_limit AND copies.size >= ctx.copy_limit AND ctx.copy_limit < total_copies %] <div class="rdetail_show_copies"> <img src="[% ctx.media_prefix %]/images/plus_sign.png[% ctx.cache_key %]" alt="[% l('Show more copies icon') %]"/> - <a href="[% mkurl('', {copy_limit => more_copies_limit, copy_offset => 0}) %]">[% l('Show more copies') %]</a> + <a href="[% mkurl('', {copy_limit => more_copies_limit, copy_offset => 0}, '', 'row' _ (copies.size + 1)) %]">[% l('Show more copies') %]</a> </div> [%- ELSIF ctx.copy_limit == more_copies_limit %] <div class="rdetail_show_copies"> ----------------------------------------------------------------------- Summary of changes: Open-ILS/src/templates-bootstrap/opac/parts/record/copy_table.tt2 | 4 ++-- Open-ILS/src/templates/kpac/parts/copy_table.tt2 | 4 ++-- Open-ILS/src/templates/opac/parts/record/copy_table.tt2 | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) hooks/post-receive -- Evergreen ILS