From 3a30d05094449d2660c8e25c953ab60bc2b74d84 Mon Sep 17 00:00:00 2001
From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Tue, 14 May 2013 15:48:39 -0400
Subject: [PATCH] OPAC Browse: i18n improvement for short terms

This makes the row of widgets "Browse by <foo> for <bar> held under
<baz>" localizable.

There is a minor usability tradeoff as we lose the clickable <label>
tags, but I think the trade is worth it.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---
 Open-ILS/src/templates/opac/browse.tt2 | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/Open-ILS/src/templates/opac/browse.tt2 b/Open-ILS/src/templates/opac/browse.tt2
index 6e806a6c63..aedce40ca1 100644
--- a/Open-ILS/src/templates/opac/browse.tt2
+++ b/Open-ILS/src/templates/opac/browse.tt2
@@ -31,20 +31,15 @@
                         <input type="hidden" name="blimit"
                             value="[% blimit %]" />
 
-                        <label for="browse-search-class">[%
-                            l('Browse by') %]</label>
-                        [% INCLUDE "opac/parts/qtype_selector.tt2"
+                        [% control_qtype = INCLUDE "opac/parts/qtype_selector.tt2"
                             id="browse-search-class" browse_only=1 %]
 
-                        <label for="browse-term">[% l('for') %]</label>
-                        <input type="text" name="bterm" id="browse-term"
-                            value="[% CGI.param('bterm') | html %]" />
-
-                        <label for="browse-context">[%
-                            l('held under') %]</label>
-                        [% INCLUDE build_org_selector id='browse-context'
+                        [% control_bterm = BLOCK %]<input type="text" name="bterm" id="browse-term"
+                            value="[% CGI.param('bterm') | html %]" />[% END %]
+                        [% control_locg = INCLUDE build_org_selector id='browse-context'
                             show_loc_groups=1
                             arialabel=l('Select holding library') %]
+                        [% l('Browse by [_1] for [_2] held under [_3]', control_qtype, control_bterm, control_locg) %]
 
                         <input type="submit" value="[% l('Go') %]" />
                     </form>
-- 
2.11.0