[Koha-patches] [PATCH] Bug 10987: Too many 'see also' and 'see from' labels occurences

Tomas Cohen Arazi tomascohen at gmail.com
Wed Oct 2 20:57:53 CEST 2013


If an authority record has more than one 4xx$a (or 5xx$a) entries the
corresponding labels 'see also' and 'used for/see from' are repeated.

To test:
- Have authority records with more than one entry on the 4xx (or 5xx) fields.
- Do a search, check "see also:" (or 'used for/see from') appear more than once.
- Apply the patch
- Reload and check it looks nicer :-D
- Repeat for: staff auth search, opac using ccsr, opac using prog
- Signoff

Regards
To+

Sponsored-by: Universidad Nacional de Cordoba
---
 .../intranet-tmpl/prog/en/css/staff-global.css     |    8 +++++---
 .../en/includes/authorities-search-results.inc     |   21 ++++++++++----------
 koha-tmpl/opac-tmpl/ccsr/en/css/opac.css           |    6 ++++--
 koha-tmpl/opac-tmpl/prog/en/css/opac.css           |    6 ++++--
 .../en/includes/authorities-search-results.inc     |   21 ++++++++++----------
 5 files changed, 33 insertions(+), 29 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
index 308e3a0..64d2f97 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
+++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
@@ -2258,11 +2258,13 @@ ul.ui-tabs-nav li {
 }
 
 .authref {
-    text-indent: 2em;
+    font-style: normal;
+    text-indent: 4em;
 }
 
-.authref .label {
-    font-style: italic;
+.seefrom, .seealso {
+    font-style: italic; 
+    text-indent: 2em;
 }
 
 #authfinderops {
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
index 10559d6..5d8819b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
@@ -14,13 +14,6 @@
             <span class="RT">RT: [% heading | html %]</span>
         [% END %]
     [% ELSE %]
-        <span class="label">
-        [% IF ( type=='seefrom' ) %]
-            used for/see from:
-        [% ELSIF  ( type=='seealso' ) %]
-            see also:
-        [% END %]
-        </span>
         <span class="heading">
         [% IF ( type=='seealso' ) %]
             [% IF ( authid ) %]
@@ -83,9 +76,11 @@
               </div>
             [% END %]
         [% ELSE %]
-            [% IF ( summary.seefrom ) %]
+            [% IF ( summary.seefrom.size > 1 ) %]
+            <div class="seefrom">
+                <span class="seefrom">used for/see from:</span>
                 [% FOREACH seefro IN summary.seefrom %]
-                    <div class="seefrom authref">
+                    <div class="authref">
                     [% PROCESS showreference
                         heading=seefro.heading
                         type=seefro.type
@@ -93,10 +88,13 @@
                     %]
                     </div>
                 [% END %]
+            </div>
             [% END %]
-            [% IF ( summary.seealso ) %]
+            [% IF ( summary.seealso.size > 1 ) %]
+            <div class="seealso">
+                <span class="seealso">see also:</span>
                 [% FOREACH seeals IN summary.seealso %]
-                    <div class="seealso authref">
+                    <div class="authref">
                     [% PROCESS showreference
                         heading=seeals.heading
                         type=seeals.type
@@ -104,6 +102,7 @@
                     %]
                     </div>
                 [% END %]
+            </div>
             [% END %]
         [% END %]
     [% END %]
diff --git a/koha-tmpl/opac-tmpl/ccsr/en/css/opac.css b/koha-tmpl/opac-tmpl/ccsr/en/css/opac.css
index 0901d6b..f0b8744 100644
--- a/koha-tmpl/opac-tmpl/ccsr/en/css/opac.css
+++ b/koha-tmpl/opac-tmpl/ccsr/en/css/opac.css
@@ -2640,11 +2640,13 @@ ul.ui-tabs-nav li {
 }
 
 .authref {
-    text-indent: 2em;
+    font-style: normal;
+    text-indent: 4em;
 }
 
-.authref .label {
+.seefrom, .seealso {
     font-style: italic;
+    text-indent: 2em;
 }
 
 .authstanza {
diff --git a/koha-tmpl/opac-tmpl/prog/en/css/opac.css b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
index c2902c1..c977b62 100644
--- a/koha-tmpl/opac-tmpl/prog/en/css/opac.css
+++ b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
@@ -2779,11 +2779,13 @@ ul.ui-tabs-nav li {
 }
 
 .authref {
-    text-indent: 2em;
+    font-style: normal;
+    text-indent: 4em;
 }
 
-.authref .label {
+.seefrom, .seealso {
     font-style: italic;
+    text-indent: 2em;
 }
 
 .authstanza {
diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/authorities-search-results.inc b/koha-tmpl/opac-tmpl/prog/en/includes/authorities-search-results.inc
index 9a2ffde..5406de3 100644
--- a/koha-tmpl/opac-tmpl/prog/en/includes/authorities-search-results.inc
+++ b/koha-tmpl/opac-tmpl/prog/en/includes/authorities-search-results.inc
@@ -13,13 +13,6 @@
             <span class="RT">RT: [% heading | html %]</span>
         [% END %]
     [% ELSE %]
-        <span class="label">
-        [% IF ( type=='seefrom' ) %]
-            used for/see from:
-        [% ELSIF  ( type=='seealso' ) %]
-            see also:
-        [% END %]
-        </span>
         <span class="heading">
         [% IF ( type=='seealso' ) %]
             [% IF ( authid ) %]
@@ -82,19 +75,24 @@
               </div>
             [% END %]
         [% ELSE %]
-            [% IF ( summary.seefrom ) %]
+            [% IF ( summary.seefrom.size > 1 ) %]
+            <div class="seefrom">
+                <span class="seefrom">used for/see from:</span>
                 [% FOREACH seefro IN summary.seefrom %]
-                    <div class="seefrom authref">
+                    <div class="authref">
                     [% PROCESS showreference
                         heading=seefro.heading
                         type=seefro.type
                     %]
                     </div>
                 [% END %]
+            </div>
             [% END %]
-            [% IF ( summary.seealso ) %]
+            [% IF ( summary.seealso.size > 1 ) %]
+            <div class="seealso">
+                <span class="seealso">see also:</span>
                 [% FOREACH seeals IN summary.seealso %]
-                    <div class="seealso authref">
+                    <div class="authref">
                     [% PROCESS showreference
                         heading=seeals.heading
                         type=seeals.type
@@ -102,6 +100,7 @@
                     %]
                     </div>
                 [% END %]
+            </div>
             [% END %]
         [% END %]
     [% END %]
-- 
1.7.9.5



More information about the Koha-patches mailing list