Search jQuery

jQuery Forum

Getting Started

gautamx07

animate scroll not reason for not working in FF

in Getting Started • 6 years ago

is $('body , html') and $('body' , 'html') two different things ?? because the below code does't work:

$('body' , 'html')
.animate({ scrollTop: ($(section_id).offset().top - $('.main-nav').height()) }, {
'duration': 500
});

but the works:

$('body , html')
.animate({ scrollTop: ($(section_id).offset().top - $('.main-nav').height()) }, {
'duration': 500
});

Replies(7)

watusiware

Re: animate scroll not reason for not working in FF

6 years ago

I would be shocked if it works in ANY browser!

jQuery() has NO calling form that accepts a plurality of arguments that are selectors. Thus, it comes at no surprise that your first example doesn't work. Your first example is invalid.

http://api.jquery.com/jQuery/

jakecigar

Re: animate scroll not reason for not working in FF

6 years ago

$('body' , 'html')

means body that is inside html. It is identical to

$('body')

In firefox you need to animate the scrollTop of html, where in all the other browser you animate the scrollTop of body. That’s why it doesn’t work!

JΛ̊KE

gautamx07

Re: animate scroll not reason for not working in FF

6 years ago

Thanks guys.

@Jake

$('body' , 'html')
means body that is inside html. It is identical to
$('body')

Í don't get how thats identical , i did't write $('html body') or $('html > body')

jakecigar

Re: animate scroll not reason for not working in FF

6 years ago

$('body' , 'html')

means find 'html', then find 'body' under it. There is only one body, and it is under html, so it is the same as

$('body')

To make FF work we always code

$('body , html').animate

Which is VERY different!

JΛ̊KE

jakecigar

Re: Re: animate scroll not reason for not working in FF

6 years ago

BTW…

$('body , html') passes one parameter

$('body' , 'html') passes two parameters.

JΛ̊KE

watusiware

Re: animate scroll not reason for not working in FF

6 years ago

That one flew past me, because I always use .find() rather than the second, context, parameter. I think it is more clear.

So, it doesn't work for a different reason than I thought it didn't work for... Doh!

jakecigar

Re: Re: animate scroll not reason for not working in FF

6 years ago

Interestingly, the second selector syntax used to be the "ever so slightly" more efficient way. It now calls the find method. So it is "ever so slightly" less efficient.

JΛ̊KE

Top Reply

Response title

Attachments

jQuery Forum

animate scroll not reason for not working in FF

Replies(7)

Re: animate scroll not reason for not working in FF

Re: animate scroll not reason for not working in FF

Re: animate scroll not reason for not working in FF

Re: animate scroll not reason for not working in FF

Re: Re: animate scroll not reason for not working in FF

Re: animate scroll not reason for not working in FF

Re: Re: animate scroll not reason for not working in FF

Statistics

Tags

Actions