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, rel_3_14 has been updated via 187390d3da4064f5c9ae50e75c74268d1b68109c (commit) from 42c72b86214c7ed37f54a4b1169b57e8494834e2 (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 187390d3da4064f5c9ae50e75c74268d1b68109c 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 53a61d38b6..e81e3db4d4 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 @@ -156,7 +156,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; -%] @@ -379,7 +379,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 < 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