dashboard: for Chrome 63+, use the faster `overscroll-behavior: conta…

…in` instead of attaching an onwheel event.
ArchiveTeam · Dec 7, 2017 · c6c5bde · c6c5bde
1 parent 70dc5cb
commit c6c5bde
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/libgrabsite/__init__.py b/libgrabsite/__init__.py
@@ -1 +1 @@
-__version__ = '1.4.4'
+__version__ = '1.4.5'
diff --git a/libgrabsite/dashboard.html b/libgrabsite/dashboard.html
@@ -187,6 +187,7 @@
 	margin: 0 0 1em 0;
 	border-radius: 3px;
 	box-shadow: 4px 4px 17px 0px #0000001c;
+	overscroll-behavior: contain;
 }
 
 .log-window-hidden {
@@ -823,7 +824,9 @@
 	// starts to scroll the page instead.  We prevent this behavior here.
 	// If the user wants to scroll the page, they need to move their
 	// mouse outside a log window first.
-	if(!isSafari) {
+	if(isChrome && getChromeMajorVersion() >= 63) {
+		// No need to attach an event; .log-window { overscroll-behavior: contain } will take care of it.
+	} else if(!isSafari) {
 		logWindowAttrs["onwheel"] = function(ev) {
 			// Note: offsetHeight is "wrong" by 2px but it doesn't matter
 			//console.log(ev, logWindow.scrollTop, (logWindow.scrollHeight - logWindow.offsetHeight));