From 1d83a422e05d58226d0be146679349c4bff2e56f Mon Sep 17 00:00:00 2001 From: Garry Collum Date: Thu, 25 Feb 2021 16:02:50 -0500 Subject: [PATCH] LP1916936-Bootstrap opac: item tags display as separate copies on small screen. This fixes the display of item tags and item notes in the bootstrap opac. It also cleans up some tags that had class defined twice. To test. 1. Add an item tag and an item note to an item. 2. In the bootstrap opac make the screen as small as possible. Notice that the display shows the tags and items as copies. 3. Apply the patch and repeat. The tags and notes should be displayed with the item that they are associated with. Signed-off-by: Garry Collum Signed-off-by: Mary Llewellyn Signed-off-by: Galen Charlton --- .../src/templates-bootstrap/opac/css/style.css.tt2 | 8 ++++ .../opac/parts/record/copy_table.tt2 | 53 ++++++++++------------ .../opac/parts/record/summary.tt2 | 15 +++--- 3 files changed, 40 insertions(+), 36 deletions(-) diff --git a/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 b/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 index 74faa1edf1..bc9cad1881 100755 --- a/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 @@ -1146,6 +1146,14 @@ div.format_icon { margin:0; } +span.copy_note_label { + font-weight: bold; +} + +span.copy_tag_type_label { + font-weight: bold; +} + #gbp_extra, #gbp_arrow_down_link { display: none; } 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 dad40fd500..489c3f2cf0 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 @@ -154,7 +154,31 @@ END; # FOREACH bib [% END %] [% callnum | html %] [% IF ctx.get_org_setting(CGI.param('loc') OR ctx.aou_tree.id, 'sms.enable') == 1 %](Text)[% END %] + {copy_id => copy_info.id}) %];rec=[%- ctx.bre_id -%]" rel="nofollow" vocab="">Text)[% END %] + + [% IF copy_info.notes; %] + [% FOREACH note IN copy_info.notes; %] +
[% note.title _ ": " | html %][% note.value | html %]
+ [% END %] + [% END %] + + [% IF copy_info.tags; %] + [% FOREACH tag IN copy_info.tags; %] +
+ [% tag.tag_type.label _ ": " | html %] + + [% IF tag.url %] + + [% END %] + [% tag.value | html %] + [% IF tag.url %] + + [% END %] + +
+ [% END %] + [% END %] + [%- IF has_parts == 'true' %] [% copy_info.part_label | html %] [%- END %] @@ -286,33 +310,6 @@ END; # FOREACH bib [%- END %] - [% IF copy_info.notes; %] - [% FOREACH note IN copy_info.notes; %] -  [% note.title | html %]: [% note.value | html %] - [% END %] - [% END %] - - [% IF copy_info.tags; %] - [% FOREACH tag IN copy_info.tags; %] - -   - - [% tag.tag_type.label _ ": " | html %] - - [% IF tag.url %] - - [% END %] - [% tag.value | html %] - [% IF tag.url %] - - [% END %] - - - - [% END %] - [% END %] - - [%- IF copy_info.peer_bib_marc.size > 0; '