I want to scroll to a div each time user presses j key. Here is the code for it.
$(function() {
function scroll(direction) {
var s
$(scroll).offset().top-50 is prefectly valid, as .top will return an integer value. Therefore the issue is not with this portion of your code.
I suspect the issue is to do with the scroll variable you have within the scroll function. I always keep away from naming my variables the same as my function names when within the same scope.