Bug 5357: Follow-up - layout and EAN search
1) EAN search should be hidden if marcflavour is not UNIMARC 2) Fixes layout to match the advanced search in acquisitions - labels in front of the fields instead of separate lines - adds a legend to the form and moves the toggle for the search form to it To test: - EAN search field should only show up when marcflavour system preference is set to UNIMARC - Check layout is consistent and you like it - Check toggle for advanced search still behaves the same Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
This commit is contained in:
parent
214fd0fbd1
commit
8defeb2bd0
2 changed files with 7 additions and 5 deletions
|
@ -37,11 +37,11 @@
|
|||
|
||||
<div id="advsearch">
|
||||
<form action="/cgi-bin/koha/serials/serials-search.pl" method="get">
|
||||
<fieldset class="brief">
|
||||
<fieldset class="rows">
|
||||
<legend onclick="$('#advsearch_form').slideToggle(400);">
|
||||
<a id="unfold_advsearch" style="cursor:pointer">Search subscriptions</a>
|
||||
</legend>
|
||||
[% IF ( done_searched ) %]
|
||||
<div onclick="$('#advsearch_form').slideToggle(400);" style="width:100%;">
|
||||
<a id="unfold_advsearch" style="cursor:pointer">Advanced search</a>
|
||||
</div>
|
||||
<div id="advsearch_form" style="display:none">
|
||||
[% ELSE %]
|
||||
<div>
|
||||
|
@ -55,10 +55,12 @@
|
|||
<label for="title">Title:</label>
|
||||
<input type="text" id="title" name="title_filter" value="[% title_filter %]" />
|
||||
</li>
|
||||
[% IF ( marcflavour == "UNIMARC" ) %]
|
||||
<li>
|
||||
<label for="ean">EAN:</label>
|
||||
<input type="text" id="ean" name="EAN_filter" value="[% EAN_filter %]" />
|
||||
</li>
|
||||
[% END %]
|
||||
<li>
|
||||
<label for="publisher">Publisher:</label>
|
||||
<input type="text" id="publisher" name="publisher_filter" value="[% publisher_filter %]" />
|
||||
|
|
|
@ -104,7 +104,7 @@ $template->param(
|
|||
branches_loop => \@branches_loop,
|
||||
done_searched => $searched,
|
||||
routing => $routing,
|
||||
(uc(C4::Context->preference("marcflavour"))) => 1
|
||||
marcflavour => (uc(C4::Context->preference("marcflavour")))
|
||||
);
|
||||
|
||||
output_html_with_http_headers $query, $cookie, $template->output;
|
||||
|
|
Loading…
Reference in a new issue