From 699b3599872761a96d535abaaf57801bd4fd4fd4 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 77c6bc2e1b..69ab8c4c7c 100755 --- a/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 @@ -1145,6 +1145,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 130d8aa028..30cc2b1e95 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 @@ -111,7 +111,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 %] @@ -240,33 +264,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; '