Attachment 5933 Details for Bug 7043 – Proposed fix

[patch] Proposed fix

0001-Fix-for-Bug-7043-masthead_search-id-used-twice-in-te.patch (text/plain), 2.12 KB, created by Owen Leonard on 2011-10-17 15:54:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2011-10-17 15:54:18 UTC

Size: 2.12 KB

patch

obsolete

>From fb3dbe82d88713b52eaf1a94230b51f807814766 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 17 Oct 2011 11:51:16 -0400
>Subject: [PATCH] Fix for Bug 7043 - masthead_search id used twice in template
>Content-Type: text/plain; charset="utf-8"
>
>Changing ID of library selector, and moving inline styles to
>the CSS as requested by kf in the bug comments.
>---
> koha-tmpl/opac-tmpl/prog/en/css/opac.css          |    6 ++++++
> koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc |    4 ++--
> 2 files changed, 8 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/prog/en/css/opac.css b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
>index 665d143..14236c8 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/css/opac.css
>+++ b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
>@@ -784,6 +784,12 @@ body#advsearch #listsmenulink {
> 	margin : .2em .3em 0 0;
> }
> 
>+#masthead_search,
>+#select_library {
>+	max-width: 10em;
>+	width: 10em;
>+}
>+
> 
> #search-facets ul {
> 	margin : 0;
>diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc b/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc
>index b3672f4..ee8b468 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc
>+++ b/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc
>@@ -35,7 +35,7 @@
> 	[% END %]
>     </label>
> 
>-	<select name="idx" id="masthead_search" class="left" style="max-width:10em;width:10em;">
>+	<select name="idx" id="masthead_search" class="left">
> 	[% IF ( ms_kw ) %]
>         <option selected="selected" value="">Library Catalog</option>
> 		[% ELSE %]
>@@ -77,7 +77,7 @@
>         <input type="text" id = "transl1" name="q" class="left" style="width: 35%; font-size: 111%;"/><div id="translControl"></div>
> [% END %]
>    [% IF ( OpacAddMastheadLibraryPulldown ) %]
>-      <select name="limit" id="masthead_search" class="left" style="max-width:10em;width:10em;">
>+      <select name="limit" id="select_library" class="left">
>          <option value="">All Libraries</option>
>          [% FOREACH BranchesLoo IN BranchesLoop %]
>             [% IF ( BranchesLoo.selected ) %]<option select="selected" value="branch:[% BranchesLoo.value %]">[% BranchesLoo.branchname %]</option>
>-- 
>1.7.3
>

Actions: View | Diff | Splinter Review

Attachments on bug 7043: 5933 | 5934