5.2 Merge request scrolls up in Chrome for windows

Created by: hithwen

Summary: When viewing a merge request and scroll down, new content is loaded below but the page scrolls up to the top every time you try to scroll down, so you cannot see the code. This happens in chrome for windows, in mac works fine. Steps to reproduce:

1. Enter a merge request
2. Scroll down
3. Wait for new content to load, page scrolls up
4. Scroll down again Expected behavior: View all page content Observed behavior: Page scrolls up all the time

Created by: dprandzioch

Same Problem in Safari on Mac OS X.

By Administrator on 2013-05-28T08:29:36 (imported from GitLab)

Created by: bbodenmiller

Thanks for well formatted issue. Is this an issue in Firefox and/or ie10?

By Administrator on 2013-05-28T14:48:11 (imported from GitLab)

Created by: hithwen

I dont see it happening in Firefox for mac, I dont have access to a windows computer now

By Administrator on 2013-05-28T14:59:49 (imported from GitLab)

Created by: dzaporozhets

What if you open page directly (not navigating via turbolinks)? Is it still scroll?

By Administrator on 2013-05-28T17:55:30 (imported from GitLab)

Created by: agb80

Same in Chromium for Linux. And regarding your question @randx, the answer is not, no bad behaivor if open page directly

By Administrator on 2013-06-04T22:24:39 (imported from GitLab)

Created by: sroth80021

This happens in Firefox 21.0 for Linux (Fedora 18) as well. Like @agb80, I do not see the issue if I open the page directly.

By Administrator on 2013-06-06T19:39:29 (imported from GitLab)

Created by: erkasraim

Same Problem in Firefox, Chrome on Mac OS X

But

1. reload(command + r) merge request page
2. fixed problem

I don't know that reload pre and post.

By Administrator on 2013-06-07T02:16:58 (imported from GitLab)

Created by: sroth80021

confirmed, I see the same -- ctrl-r is a workaround (though not obvious) for this issue. Thanks!

By Administrator on 2013-06-07T04:59:26 (imported from GitLab)

Created by: dzaporozhets

I fixed it for master

By Administrator on 2013-06-07T06:22:29 (imported from GitLab)

Created by: dzaporozhets

will be in 5.3 release

By Administrator on 2013-06-07T06:22:47 (imported from GitLab)