Archiveteam/grab-site
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use display: none on log windows in FF to avoid perf issues

Browse Source
This commit is contained in:
Ivan Kozik 2015-03-27 13:45:03 +00:00
parent 486453d63b
commit fe9a552385
1 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
dashboard/dashboard.html
View File

@ -470,6 +470,7 @@ var getQueryArgs = function() {
};
var isSafari = navigator.userAgent.indexOf("Safari") != -1;
var isFirefox = navigator.userAgent.indexOf("Firefox") != -1;
var addAnyChangeListener = function(elem, func) {
// DOM0 handler for convenient use by Clear button
@ -1004,16 +1005,27 @@ JobsRenderer.prototype.applyFilter = function() {
var w = this.renderInfo[job["ident"]].logWindow;
if(!RegExp(query).test(job["url"])) {
w.classList.add("log-window-hidden");
// Remove this class, else an ugly border will be visible
// Firefox exhibits serious performance problems when adding
// lines to our 0px-high log windows, so add display: none
// (effectively killing the animation)
if(isFirefox) {
w.style.display = "none";
}
// Remove this class, else an ugly border may be visible
w.classList.remove('log-window-stopped');
unmatchedWindows.push(w);
} else {
w.classList.remove("log-window-hidden");
if(isFirefox) {
w.style.display = "block";
}
matches += 1;
matchedWindows.push(w);
if(this.firstFilterMatch == null) {
this.firstFilterMatch = job["ident"];
}
w.classList.remove("log-window-hidden");
matches += 1;
matchedWindows.push(w);
}
}