Collapsing/expanding in pushloghtml is extremely slow when there are a lot of changesets per push



4 years ago
4 years ago


(Reporter: glandium, Unassigned)





4 years ago
Try collapsing a push. I've given up and stopped the script after getting the script busy popup five times. 1638 table rows is nowhere enough to make something /that/ slow. It must be doing something in O(n^2) or worse. And debugging minified jquery code is painful.

In fact, using the developer tools to add a tr.hidden.id86 { display:none; } CSS rule, which would be the equivalent to collapsing the first push (as of writing) only takes a couple seconds, compared to the minutes it took with the script before I gave up.

It could probably be even faster if the html structure was changed, but changing the script to do the CSS thing instead of whatever it's currently doing should be simple enough.
You need to log in before you can comment on or make changes to this bug.