Bug 8884: opacheader is not in the header on CCSR
authorJared Camins-Esakov <jcamins@cpbibliography.com>
Mon, 20 May 2013 16:55:17 +0000 (12:55 -0400)
committerGalen Charlton <gmc@esilibrary.com>
Sun, 30 Jun 2013 15:32:48 +0000 (08:32 -0700)
In CCSR the opacheader preference isn't really in the header it's to the
right of the search box (see attached). I'm okay with a customizable
region over there, but we should have a new preference for that and
header should be above the search box as it implies.

To test:
1) Put something in the OpacHeader syspref. Preferably something large
   and header-like for maximum effect.
2) Select CCSR theme.
3) View OPAC. Notice placement of "header" content.
4) Apply patch.
5) View OPAC. Notice placement of "header" content.
6) Sign off.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Passes koha-qa.pl, works as advertised with one caveat; the content of
opacheader is by default underneath the div "top-bar". A couple breaks
in my OpacHeader test made it visible, so I'm going to assume this is a
styling issue and not a bug as such.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/opac-tmpl/ccsr/en/includes/masthead.inc

index ec699f4..0069f74 100644 (file)
 
 </div>
 
+<div id="opac-custom-header">
+[% IF ( opacheader ) %]
+    [% opacheader %]
+[% END %]
+</div>
+
 <div id="opac-main-search" class="yui-g">
 
 <a class="logo" href="/cgi-bin/koha/opac-main.pl">
 </div>
 
 <div id="libraryname">
-    [% IF ( opacheader ) %]
-        <h1>
-            [% opacheader %]
-        </h1>
-  [% END %]
+    <h1></h1>
 </div>
 
 [% END %]  <!-- OpacPublic -->