From 37eaab328a17035f69119bf4a98cff41e9972904 Mon Sep 17 00:00:00 2001 From: Lebbeous Fogle-Weekley Date: Fri, 20 Jan 2012 18:28:50 -0500 Subject: [PATCH] Amazingly, only one weird bug broke IE8, and now it's worked around It's a stupid one though (of course). See comments in code. Signed-off-by: Lebbeous Fogle-Weekley --- Open-ILS/web/opac/skin/default/js/search_bar.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Open-ILS/web/opac/skin/default/js/search_bar.js b/Open-ILS/web/opac/skin/default/js/search_bar.js index 5f8cdb9eab..9a86edede0 100644 --- a/Open-ILS/web/opac/skin/default/js/search_bar.js +++ b/Open-ILS/web/opac/skin/default/js/search_bar.js @@ -48,6 +48,12 @@ function updateSearchTypeSelector(id) { } function autoSuggestInit() { + /* For the "style" argument to the constructor below, IE 8 hates it if + * we supply dojo.attr("search_box", "style") as the value, which worked + * nicely in Google Chrome and Firefox. Filtering the "font" key/value + * pair out of that object gets IE not to throw an error, but even then + * the result is oddly ugly borders. So for now we're sadly hardcoding + * style. */ var widg = new openils.widget.AutoSuggest( { "storeArgs": { @@ -62,7 +68,7 @@ function autoSuggestInit() { if (event.charOrCode == dojo.keys.ENTER) searchBarSubmit(); }, - "style": dojo.attr("search_box", "style"), + "style": {"width": "260px"}, "value": ((getTerm() != null) ? getTerm() : "") }, "search_box" ); -- 2.11.0