Skip to content

Commit 824e6d6

Browse files
committed
highlighter: remove redundant (?) call to $.show()
I don't understand why this would be necessary -- the element's just been created, no style is (presumably) changing its display to anything other than the default... -- and when highlighting all text in a 5000×3 table, $.show() accounts for 10s out of the 14s it takes to draw the highlights. $.show() being absurdly slow is a known problem: jquery/jquery.com#88
1 parent 5015d2d commit 824e6d6

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/ui/highlighter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ function highlightRange(normedRange, cssClass) {
3434
var node = nodes[i];
3535
if (!white.test(node.nodeValue)) {
3636
results.push(
37-
$(node).wrapAll(hl).parent().show()[0]
37+
$(node).wrapAll(hl).parent()[0]
3838
);
3939
}
4040
}

0 commit comments

Comments
 (0)