From: Garry Collum Date: Tue, 14 Dec 2021 19:38:34 +0000 (-0500) Subject: LP1903767 - Bootstrap Opac: Make other formats and editions more visible. X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=a0da4d794d19d9ea5c92c2ffaf7c6d92713cf62b;p=working%2FEvergreen.git LP1903767 - Bootstrap Opac: Make other formats and editions more visible. This patch does the following: 1. Moves "Other Formats and Editions" below # of holds and available items. 2. Changes "Other Options" to "Other Formats and Editions" and made the header tag and

to be consistent with the other header tags. 3. Resizes

headers. 4. Removes bullets and changes the margins of some of the table/list displays under the

headers. 5. Fixes the alignment of the button group (Place Hold, Remove from basket) For testing: 1. Find a some records with other formats and editions. In concerto record #71 contains other formats and editions. You could also copy a couple of existing records and make some changes to create other formats and editions. For example, add a language code, or change the publication date. 2. Notice that you have to click on the More Details button in order to view the Other Formats. 3. Apply the patch. 4. Other Formats and Editions now appears under the Available Copies and Holds Headings. Signed-off-by: Garry Collum --- 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 a40c8eb28b..668dc2aef0 100755 --- a/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 @@ -925,6 +925,11 @@ div#rdetail_actions_div { [%- END %] } +div#rdetail_copy_counts { + margin-top: 1em; +} + +/* span#rdetail_copy_counts { display: inline-block; vertical-align: top; @@ -950,6 +955,7 @@ span#rdetail_hold_counts p { padding-left: 2em; [%- END %] } +*/ #rdetail_image_div { [% IF rtl == 't' -%] @@ -2403,12 +2409,30 @@ a.opac-button-header:hover, #dash_wrapper a.opac-button:hover { width: 100%; } -#rdetail_record_details { +.rdetail_record_details { clear: both; margin-top: 1em; } -.rdetail_content { +h2.rdetail_record_details, #rdetail_copy_counts h2, h2.rdetail_contents, +h2.rdetail_related_subjects, h2.rdetail_related_series, +h2.rdetail_other_formats, h2.rdetail_uris { + font-size: [% css_fonts.size_biggest %]; + font-weight: bold; +} + +ul.no-bullets { + list-style-type: none; + [% IF rtl == 't' -%] + margin-right: 0; + padding-right: 1.5em; + [% ELSE -%] + margin-left: 0; + padding-left: 1.5em; + [% END -%] +} + +.rdetail_content, .rdetail_subject { [% IF rtl == 't' -%] margin-right: 1.5em; padding-right: 1.5em; diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_counts.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_counts.tt2 index 407662d50b..24f57975a0 100755 --- a/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_counts.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/parts/record/copy_counts.tt2 @@ -1,7 +1,7 @@

[% l('Available copies') %]

-
    +
      [%- depths = ctx.copy_summary.size; depth = 0; displayed_ous = {}; @@ -52,7 +52,7 @@

    [% l('Holds') %]

    -
    • +
      • [% # If org hiding is enabled/relevant, only show # counts for copies within the hiding scope. diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/record/series.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/record/series.tt2 index ea93d55e0b..cb484d4aef 100755 --- a/Open-ILS/src/templates-bootstrap/opac/parts/record/series.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/parts/record/series.tt2 @@ -59,7 +59,7 @@ END; [%- series_anchors = PROCESS render_series; IF series_anchors.length > 0; %]

        [% l('Search for related items by series') %]

        -
          +
            [% series_anchors %]
          [%- END %] diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2 index a0007ed16f..affb01db29 100755 --- a/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2 +++ b/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2 @@ -73,7 +73,7 @@ ctx.metalinks.push('
          -

          [% l("Record details") %]

          +

          [% l("Record details") %]

            [%- IF attrs.hl.isbn.size; FOR isbn IN attrs.hl.isbn %]
          • @@ -254,6 +254,30 @@ ctx.metalinks.push(' [%- INCLUDE "opac/parts/record/contents.tt2" %] [%- INCLUDE "opac/parts/record/subjects.tt2" %] [%- INCLUDE "opac/parts/record/series.tt2" %] + +
          + +
+ + +
+ + [%- + # Hold/copy summary + IF ctx.copy_summary.0.count; + INCLUDE "opac/parts/record/copy_counts.tt2"; + %] + + + [%- END %] + +
[%- # l( 'mmr id = ' _ ctx.mmr_id ); @@ -263,7 +287,7 @@ ctx.metalinks.push(' IF args.mmr_unique_bib.size > 1; %] -

[% l("More Options") %]

+

[% l("Other Formats and Editions") %]

[%- IF mmr_attrs.format_label; @@ -320,35 +344,11 @@ ctx.metalinks.push(' [%- END; END # ending tag for IF args.mmr_unique_bib.size > 1; %]
- - - - - - -
- - [%- - # Hold/copy summary - IF ctx.copy_summary.0.count; - INCLUDE "opac/parts/record/copy_counts.tt2"; - %] - - - [%- END %] - -
-
+
[%- search_ou = ctx.search_ou; IF ctx.place_unfillable || ( attrs.marc_xml.findnodes('//*[local-name()="holdings" and @has_holdable="true"]').size @@ -358,7 +358,7 @@ ctx.metalinks.push(' - [% l('Place Hold') %] + [% l('Place Hold') %] [%- END -%]