Page rescrolling to top in some browsers when Javascript runs

Please refer to this test page:

If you scroll down, there is a list of links to the left (Beef, Black Angus, etc.). If you click those, the color of the link changes and the content to the right changes. That is handled with this script:

$('#menu3 li').on('click', function(){
	var menu3click=$(this).index();
	$('#menu3 li').removeClass('onpage');
	if (!$('div#subcontent'+menu3click).is(':visible')){
		$('div.subcontent:visible').fadeOut(200, function(){

Open in new window

They are not really links, though, the code looks like this:

<ul id="menu3">
	<li>HOT DOGS</li>
	<li>Black Angus</li>

Open in new window

On all of my browsers, this is working fine (Windows/Mac, IE, Firefox, Safari, Chrome). However, my client (who is running Safari on a MacBook Pro) claims that every time he clicks one of those "links", his browser rescrolls all the way to the top of the document, which is what we are trying to avoid with these animated content swaps. I have had him clear his cache, restart browser, etc., but still not working for him. He says, "the movement is abrupt, the page is not reloading", it just seems to bounce back up to the top of the document.

I don't see anything in my code that would cause this, as its all handled by Javascript. I would appreciate any advice as I don't know what to do, since I can't even duplicate the "problem".

Thank you!
Brad BansnerWeb DeveloperAsked:
Who is Participating?
GaryConnect With a Mentor Commented:
Yes - put the col resizer code in a function that you can call after you have faded in the content ($('div#subcontent'+menu3click).fadeIn(200))
And a call to the function to run at page load.
What's happening is that when you click the link and remove the content the page is shortened, the new content is added but the scroll position is being set at the no content position - which is somewhere in the middle of your hot dog
(If that makes sense)

I will have to think about a solution.
One possible solution is wrap all your subcontentn divs in one container, and add this container into your column resizing so it is always present on the page.
Cloud Class® Course: Ruby Fundamentals

This course will introduce you to Ruby, as well as teach you about classes, methods, variables, data structures, loops, enumerable methods, and finishing touches.

Brad BansnerWeb DeveloperAuthor Commented:
I don't really understand, since the menu on the left side is taller than the content area anyway (or at least around the same height). As I said, I can't even duplicate the behavior in any of my browsers. Can you?
In Safari scroll to the bottom and click the links.
Your links are not the same height, it is shorter than the content.
Brad BansnerWeb DeveloperAuthor Commented:
In my script that resizes the content areas (on page load), I added this:


Open in new window

That might help with this? It will set the height of the left menu to the same as the maximum right-side content.

Not sure if this:

var getmaxheight=Math.max($('div.content3div').eq(0).height(),$('div.content3div').eq(1).height());

Open in new window

…only calculates the first instance of the content DIVs. I'm assuming it will ignore all the hidden ones that are setup for content swap when the menu is clicked.

I might need to re-run that calculation each time a menu item is clicked?
Brad BansnerWeb DeveloperAuthor Commented:
My client reports: "it doesn't jump as much". Maybe this is as good as it gets, it still need to resize if the secondary content is taller than the first, right?
Yes, its about the best you can get. Though it shouldn't be jumping at all if the content is the same or smaller.
Brad BansnerWeb DeveloperAuthor Commented:
Thanks for your help!
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.