Closed
Bug 1243554
Opened 9 years ago
Closed 9 years ago
Trackpad scrolling on Mac is very janky, sometimes, in CodeMirror
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
RESOLVED
FIXED
mozilla47
People
(Reporter: dietrich, Assigned: kats)
References
Details
(Keywords: regression, Whiteboard: [gfx-noted])
See original comment at: https://bugzilla.mozilla.org/show_bug.cgi?id=1160601#c9
WebIDE editor on Mac is unusable when scrolling with the trackpad.
Easy to reproduce by loading even a small JS file in the WebIDE editor and scrolling using two fingers on the trackpad.
Video where I am attempting to scroll for the entire duration:
https://www.youtube.com/watch?v=z6d0402uWPU
I turned off APZ and the bug still occurs.
See also bug 1239615 which is about APZ. I'm filing this separately since I'm seeing with and without APZ. Feel free to dupe it if you want.
I believe this related to APZ and CodeMirror in general, not something WebIDE is doing. Moving to APZ for now.
Component: Developer Tools: WebIDE → Panning and Zooming
Product: Firefox → Core
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #1)
> I believe this related to APZ and CodeMirror in general, not something
> WebIDE is doing. Moving to APZ for now.
Per comment 0 this happens with APZ disabled as well, so unless you have some justification for your belief, this is not really acceptable. For the moment I'm willing to mark this dependent on bug 1239615 and see if fixing that helps here, but if it doesn't I'm going to throw it back to you.
Depends on: 1239615
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #2)
> (In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #1)
> > I believe this related to APZ and CodeMirror in general, not something
> > WebIDE is doing. Moving to APZ for now.
>
> Per comment 0 this happens with APZ disabled as well, so unless you have
> some justification for your belief, this is not really acceptable. For the
> moment I'm willing to mark this dependent on bug 1239615 and see if fixing
> that helps here, but if it doesn't I'm going to throw it back to you.
Makes sense. I just can't think of anything that would be specific to WebIDE in particular, so hopefully fixing bug 1239615 fixes this too.
Assignee | ||
Comment 4•9 years ago
|
||
Making title less scary.
Summary: trackpad scrolling on Mac is broken → trackpad scrolling on Mac is broken, sometimes, in CodeMirror
Updated•9 years ago
|
Whiteboard: [gfx-noted]
Blocks: 1246306
For this bug, which seems to be both APZ on or off, let's see if a regression window can help.
Keywords: regressionwindow-wanted
OS: Unspecified → Mac OS X
Summary: trackpad scrolling on Mac is broken, sometimes, in CodeMirror → Trackpad scrolling on Mac is broken, sometimes, in CodeMirror
I'm unable to reproduce this bug locally. Dietrich, please see if you can find the regression window.
Flags: needinfo?(dietrich)
Reporter | ||
Comment 7•9 years ago
|
||
I'm not going to be able to do that anytime soon. Can confirm is still unusably bad in Nightly, not magically fixed yet.
Flags: needinfo?(dietrich)
(In reply to Dietrich Ayala (:dietrich) from comment #7)
> I'm not going to be able to do that anytime soon.
Well then, can you provide any more information that will enabled me to reproduce this locally? Unless I am able to a regression window will be impossible.
Reporter | ||
Comment 9•9 years ago
|
||
Hm, I'm on the 12" Macbook. I just reproduced instantly on Dev Edition, Nightly, and Firefox. Maybe hardware?
(In reply to Dietrich Ayala (:dietrich) from comment #7)
> I'm not going to be able to do that anytime soon. Can confirm is still
> unusably bad in Nightly, not magically fixed yet.
Dietrich, could you check in Release (44) as well? I can try to find a regression, but I am starting to think this has been around longer than we guessed. Just want to see if you get a different result.
For me, it seems to be present in Release as well.
Flags: needinfo?(dietrich)
Reporter | ||
Comment 11•9 years ago
|
||
Yep, see comment #9 :)
Interestingly, it is not as bad in 44... I'm not seeing the screen whiteout effect that I see when scrolling up and down quickly in Nightly.
Flags: needinfo?(dietrich)
Comment 12•9 years ago
|
||
Could you try the patch from Bug 1243589 ?
It completely fixes the issue logged in Bug 1239615. I quickly tried WebIDE with it, and it seems to improve the situation as well.
Assignee | ||
Updated•9 years ago
|
status-firefox44:
--- → affected
status-firefox45:
--- → affected
status-firefox46:
--- → affected
status-firefox47:
--- → affected
Keywords: regressionwindow-wanted
Assignee | ||
Updated•9 years ago
|
Summary: Trackpad scrolling on Mac is broken, sometimes, in CodeMirror → Trackpad scrolling on Mac is very janky, sometimes, in CodeMirror
Assignee | ||
Comment 13•9 years ago
|
||
Please retest on the next Nightly; bug 1243589 should fix this or at least improve it significantly. Thanks!
Flags: needinfo?(dietrich)
Reporter | ||
Comment 14•9 years ago
|
||
It's WAY better. The interaction is smooth now - the motion of the scroll is exactly matching my physical scrolling movement. However, visually it is still not great, it's skipping bits, looks like. I'll try and make a movie showing it.
Flags: needinfo?(dietrich)
Assignee | ||
Comment 15•9 years ago
|
||
(In reply to Dietrich Ayala (:dietrich) from comment #14)
> It's WAY better. The interaction is smooth now - the motion of the scroll is
> exactly matching my physical scrolling movement. However, visually it is
> still not great, it's skipping bits, looks like. I'll try and make a movie
> showing it.
If by "skipping bits" you mean it looks like checkerboarding (i.e. parts of the code window are blank for a while but fill in later) then it might be https://github.com/codemirror/CodeMirror/issues/3564 (which bug 1232491 would help with). Still a video would help confirm we're talking about the same thing.
Reporter | ||
Comment 16•9 years ago
|
||
https://www.youtube.com/watch?v=ThXBlhYhR_E
The latter part of the video is where the effect is visible - looks like a flashing on both the top and bottom of the scrolling area.
Assignee | ||
Comment 17•9 years ago
|
||
A bit hard to tell but that looks like a combination of APZ checkerboarding and what I described in comment 15. I'll mark this fixed by bug 1243589 since the jank issue is fixed, and file a follow-up to track the checkerboarding.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Updated•9 years ago
|
Updated•8 years ago
|
Assignee: nobody → bugmail
Target Milestone: --- → mozilla47
Version: unspecified → 44 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•