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_15 has been updated via 110f0378c71c1c5e816afb04798822c01d1385fa (commit) from 87e7ecad910455eab0e2bc26e980debf29e60aaa (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 110f0378c71c1c5e816afb04798822c01d1385fa 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