Page rescrolling to top in some browsers when Javascript runs

Brad Bansner
Brad Bansner used Ask the Experts™
on
Please refer to this test page:
http://www.berksfoods.com/retail-hot-dogs-test.htm

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');
	$(this).addClass('onpage');
	if (!$('div#subcontent'+menu3click).is(':visible')){
		$('div.subcontent:visible').fadeOut(200, function(){
			$('div#subcontent'+menu3click).fadeIn(200);
		});
	}
});

Open in new window


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

<ul id="menu3">
	<li>HOT DOGS</li>
	<li>Beef</li>
	<li>Black Angus</li>
	<li>Original</li>
	<li>Grill</li>
	<li>Jumbo</li>
	<li>Cheese</li>
	<li>Fiesta</li>
	<li>Pretzel</li>
</ul>

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!
Comment
Watch Question

Do more with

Expert Office
EXPERT OFFICE® is a registered trademark of EXPERTS EXCHANGE®
Expert of the Year 2014
Top Expert 2014

Commented:
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.
Expert of the Year 2014
Top Expert 2014

Commented:
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.
Brad BansnerWeb Developer

Author

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?
Ensure you’re charging the right price for your IT

Do you wonder if your IT business is truly profitable or if you should raise your prices? Learn how to calculate your overhead burden using our free interactive tool and use it to determine the right price for your IT services. Start calculating Now!

Expert of the Year 2014
Top Expert 2014

Commented:
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 Developer

Author

Commented:
In my script that resizes the content areas (on page load), I added this:

$('#menu3').height(getmaxheight);

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?
Expert of the Year 2014
Top Expert 2014
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.
Brad BansnerWeb Developer

Author

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?
Expert of the Year 2014
Top Expert 2014

Commented:
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 Developer

Author

Commented:
Thanks for your help!

Do more with

Expert Office
Submit tech questions to Ask the Experts™ at any time to receive solutions, advice, and new ideas from leading industry professionals.

Start 7-Day Free Trial