From 284affaf5c185b530374b9c6d65b3419d74a53f3 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 17 Jan 2013 16:52:18 +0100 Subject: [PATCH] Bug 9419: The highlight feature can cause a browser freeze Content-Type: text/plain; charset="utf-8" Test plan: - switch on the OpacHighlightedWords syspref - search at the opac and intranet a string with a space (e.g. au,wrdl:"Shakespeare, William" note the double space after the comma) - without the patch your browser will be frozen during a few seconds, after applying it, the highlight works better. Signed-off-by: Kyle M Hall Signed-off-by: Marcel de Rooy --- .../prog/en/modules/catalogue/results.tt | 4 +++- .../opac-tmpl/prog/en/modules/opac-results.tt | 8 +++++--- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt index b5035c9..1b20b07 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt @@ -134,7 +134,9 @@ $('#sort_by').change(function() { function highlightOn() { var x; for (x in q_array) { - toHighlight.highlight(q_array[x]); + if ( q_array[x].length > 0 ) { + toHighlight.highlight(q_array[x]); + } } $(".highlight_toggle").toggle(); }[% END %] diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt index 0986616..9f0b044 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt @@ -95,9 +95,11 @@ function highlightOff() { function highlightOn() { var x; for (x in q_array) { - $(".title").highlight(q_array[x]); - $(".author").highlight(q_array[x]); - $(".results_summary").highlight(q_array[x]); + if ( q_array[x].length > 0 ) { + $(".title").highlight(q_array[x]); + $(".author").highlight(q_array[x]); + $(".results_summary").highlight(q_array[x]); + } } $(".highlight_toggle").toggle(); } -- 1.7.7.6