Fixes scrollIntoView when scrollbars are hidden

yurydelendik · yurydelendik · commit c53b7aacfc01 · 2012-08-28T16:29:47.000-05:00
diff --git a/web/viewer.js b/web/viewer.js
@@ -38,6 +38,8 @@ function scrollIntoView(element, spot) {
   while (parent.clientHeight == parent.scrollHeight) {
     offsetY += parent.offsetTop;
     parent = parent.offsetParent;
+    if (!parent)
+      return; // no need to scroll
   }
   if (spot)
     offsetY += spot.top;

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,8 @@ function scrollIntoView(element, spot) {`
`38`	`38`	`while (parent.clientHeight == parent.scrollHeight) {`
`39`	`39`	`offsetY += parent.offsetTop;`
`40`	`40`	`parent = parent.offsetParent;`
	`41`	`+ if (!parent)`
	`42`	`+ return; // no need to scroll`
`41`	`43`	`}`
`42`	`44`	`if (spot)`
`43`	`45`	`offsetY += spot.top;`