My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 3090: SideBySide2: page stops responding after scrolling on empty side
1 person starred this issue and may be notified of changes. Back to list
Status:  Accepted
Owner:  sop@google.com


Sign in to add a comment
 
Project Member Reported by david.pu...@sonymobile.com, Jan 6, 2015
************************************************************
***** NOTE: THIS BUG TRACKER IS FOR GERRIT CODE REVIEW *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, INTERNAL *****
***** ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.    *****
***** THOSE ISSUE BELONG IN DIFFERENT ISSUE TRACKERS!  *****
************************************************************

Affected Version: 2.10-rc1-1211-g04c4f09

What steps will reproduce the problem?
1. Open side-by-side diff on a file that was added, for example [1]
2. Hover the mouse over the right side (the added file) and use the mouse wheel to scroll down to the bottom of the file
3. Move the mouse over the left side (blank) and use the mouse wheel to scroll so that the red scroll bar on the left hand side (with '1' at the bottom) moves up.

What is the expected output? What do you see instead?

When the red scroll bar and '1' reaches the top of the screen, the page stops responding to scrolls, clicks and key presses. The tab has to be closed and the page must be reloaded.

Same occurs when scrolling on the right side (empty) of diff for a file that has been removed, for example [2]

Please provide any additional information below.

[1] https://gerrit-review.googlesource.com/#/c/62912/4/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/AddFileAction.java

[2] https://gerrit-review.googlesource.com/#/c/62912/4/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/EditFileBox.java
Jan 6, 2015
#1 sop@google.com
It also kills the browser tab. Chrome stopped responding and I had to kill that tab.

This is probably something in the interaction between CodeMirror and our ScrollSynchronizer.
I'll try to look at it more tomorrow.
Status: Accepted
Owner: sop@google.com
Cc: -sop@google.com
Sign in to add a comment

Powered by Google Project Hosting