Bug 9419: The highlight feature can cause a browser freeze

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 <kyle@bywatersolutions.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
This commit is contained in:
Jonathan Druart 2013-01-17 16:52:18 +01:00 committed by Jared Camins-Esakov
parent 9867d3c64d
commit 3e413c5491
2 changed files with 8 additions and 4 deletions

View File

@ -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 %]

View File

@ -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();
}