Open Bug 1331390 Opened 4 years ago Updated 9 days ago

Over scrolling when mouse down on scrollbar if smooth scroll is enabled

Categories

(Core :: Layout, defect, P3)

34 Branch
All
Windows 10
defect

Tracking

()

Tracking Status
firefox50 --- wontfix
firefox51 --- wontfix
firefox52 --- fix-optional
firefox53 --- wontfix
firefox54 --- fix-optional

People

(Reporter: alice0775, Assigned: kip)

References

Details

(5 keywords)

Build Identifier:
https://hg.mozilla.org/mozilla-central/rev/5ce3882eec21be3a70e4afc050959ca2f76bfa76
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0 ID:20170115030210

The problem is noticeable on long page. 
Over scrolling when mouse down on scrollbar. It is not intuitive behavior.
Disabled smooth scroll fix the problem.

Reproducible: always

Steps To Reproduce:
1. Open long page (e.g., https://developer.mozilla.org/en-US/docs/Web/Events )
2. Mouse down and keep on the empty area of vertical scrollbar and do not move mouse pointer
3. wait to stop scroll
   --- observe page scroll position, and thumb position
4. Mouse up and click (also do not move mouse pointer)
   --- observe page scroll position, and thumb position

Actual Results:
Page is over scrolling.
The thumb is overshoot beyond position the mouse pointer.
After Step4, scroll position of page is going back.

Expected Results:
Page should not be over scrolling. (Edge and Chrome does).
After Step4, scroll position of page should not be going back.

Regression window:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=ec4622e6e1fa&tochange=4ef03db1385b

Regressed by: Bug 957445
Maybe the same underlying issue as in bug 1328053?
Flags: needinfo?(kgilbert)
Indeed, this may be the same root cause.  I am adding a patch to Bug 1328053, which may fix this as well.  Once Bug 1328053 lands, please re-test to see if it is fully fixed.
Depends on: 1328053
Flags: needinfo?(kgilbert)
I have pushed a series of try builds out for the proposed fix to Bug 1328053.  Hopefully we can use these builds to verify that it fixes this bug also.
I've reproduced this (Windows 10, FF Nightly).  It appears the fix to Bug 1328053 does not correct it.
Still reproducible, but I think this is edge-case enough that I'm calling it fix-optional for 52. That said, we still have time to take a low-risk fix for 52 if you have more time to look into this, Kip :)
Flags: needinfo?(kgilbert)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #5)
> Still reproducible, but I think this is edge-case enough that I'm calling it
> fix-optional for 52. That said, we still have time to take a low-risk fix
> for 52 if you have more time to look into this, Kip :)

I won't likely have time to investigate this one before the trains run, but would be glad to take a look for the following release.

I'll take the bug for now, but please feel free to steal this from me if anyone wants to get it in for 52.
Assignee: nobody → kgilbert
Flags: needinfo?(kgilbert)
Too late for a fix for 53, fix-optional for 54, minor carryover regression.
Duplicate of this bug: 1369847
Priority: -- → P3
Duplicate of this bug: 1423240
Mass bug change to replace various 'parity' whiteboard flags with the new canonical keywords. (See bug 1443764 comment 13.)
Whiteboard: [DUPEME],[parity-Chrome][parity-Edge] → [DUPEME],
Whiteboard: [DUPEME], → [DUPEME]
Keywords: dupeme
Whiteboard: [DUPEME]
See Also: → 1686414
You need to log in before you can comment on or make changes to this bug.