>return loWindow.pageYOffset || document.documentElement.scrollTop || document.body.scrollTop || 0; >Best practive, at least on this side, is to always have the return line to return one value and not execute some logic. Also, this syntax differs too much from other environments and we have structure in place to develop code in the same way. That is why, as you may have seen by now, that having a well understable code on multiple lines is preferred. I also really do not want anyone on the team to try to develop an entire app on one single line, sort of, but you get my point, which is something I have seen at several occasions, making everything so difficult to understand, so difficult to maintain and so difficult to evolve.