Closed
Bug 1030137
Opened 10 years ago
Closed 10 years ago
Drag to scroll using touch input is broken with e10s on desktop
Categories
(Firefox :: General, defect, P3)
Tracking
()
People
(Reporter: superbollos, Assigned: jimm)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.3; Trident/7.0; Touch; .NET4.0E; .NET4.0C; .NET CLR 3.5.30729; .NET CLR 2.0.50727; .NET CLR 3.0.30729; Tablet PC 2.0; InfoPath.3; BRI/2; MDDRJS; rv:11.0) like Gecko
Steps to reproduce:
On touchscreen drag to scroll is broken
Comment 1•10 years ago
|
||
Reproduced on Nightly 33.0a1 2014-06-29 using an Ultrabook with Win 8 64-bit.
Status: UNCONFIRMED → NEW
Component: Untriaged → General
Ever confirmed: true
Summary: Drag to scroll broken with Electrolysis → [e10s] Drag to scroll broken with Electrolysis
Comment 2•10 years ago
|
||
Daniel: when you say "touchscreen drag to scroll is broken", does it not scrolling it all? Or does it scroll but incorrectly?
Blocks: core-e10s
tracking-e10s:
--- → ?
Comment 3•10 years ago
|
||
(In reply to Chris Peterson (:cpeterson) from comment #2)
> Daniel: when you say "touchscreen drag to scroll is broken", does it not
> scrolling it all? Or does it scroll but incorrectly?
Instead of scrolling, the text your finger is above just gets selected.
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Summary: [e10s] Drag to scroll broken with Electrolysis → Drag to scroll using touch input is broken with e10s on desktop
Assignee | ||
Updated•10 years ago
|
Priority: -- → P3
Updated•10 years ago
|
Flags: firefox-backlog+
Updated•10 years ago
|
Flags: firefox-backlog+
Updated•10 years ago
|
Flags: firefox-backlog+
Comment 4•10 years ago
|
||
Move old M2 P3 bugs to M4 (because tracking-e10s=m5+ flag doesn't exist yet).
Comment 5•10 years ago
|
||
Reproduced 35.0a1 (2014-09-14) - MS Surface Pro (1) - Windows 8.1 Pro/64Bit
Cannot use touch to scroll either on main pane or using scrollbar.
Comment 8•10 years ago
|
||
This bug is very annoying. I'm disabling e10s in the meanwhile.
Blocks: win-touch-issues
Comment 9•10 years ago
|
||
Hi, I'm on 37.0a1 (2014-12-25)64-bit, on 64-bit Windows 8.1. I've noticed that scrolling works properly when using two fingers instead of just one.
Comment 10•10 years ago
|
||
I'm unable to test electrolysis on several of my systems until this is fixed, but I'm struggling to find info on what/when these milestones are, so that I know when to jump back in. Does M2.3 from here (https://wiki.mozilla.org/Electrolysis/Roadmap) = M5? And which of those milestones is the current one?
Assignee | ||
Comment 11•10 years ago
|
||
(In reply to voracity from comment #10)
> I'm unable to test electrolysis on several of my systems until this is
> fixed, but I'm struggling to find info on what/when these milestones are, so
> that I know when to jump back in. Does M2.3 from here
> (https://wiki.mozilla.org/Electrolysis/Roadmap) = M5? And which of those
> milestones is the current one?
I've just updated https://wiki.mozilla.org/Electrolysis and https://wiki.mozilla.org/Electrolysis/Roadmap with a current set of milestones.
Assignee | ||
Comment 12•10 years ago
|
||
This is mostly there, except I need a more reliable way of triggering calls to UpdateParentScrollInfo.
Assignee | ||
Comment 13•10 years ago
|
||
This is a work around to get touch scroll working with e10s. It's not clear if we'll ever have the data to fix this right but Apz frame metrics data and touch region data might be useful down the road.
Attachment #8561567 -
Attachment is obsolete: true
Attachment #8562057 -
Flags: review?(bugs)
Comment 14•10 years ago
|
||
Comment on attachment 8562057 [details] [diff] [review]
patch
> bool displayPanFeedback = false;
> for (nsIFrame* current = targetFrame; current;
> current = nsLayoutUtils::GetCrossDocParentFrame(current)) {
>
>+ // e10s - mark remote content as draggable.
Not draggable. Pannable perhaps?
in this method there is
"If that's the case, the current touch gesture
* will be used as a pan gesture; otherwise it will be a regular
* mousedown/mousemove/click event."
so I assume that comment is just bogus, since we certainly don't want to prevent mousedown/move/click events.
Can't test that. Please do test.
Attachment #8562057 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 15•10 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #14)
> in this method there is
> "If that's the case, the current touch gesture
> * will be used as a pan gesture; otherwise it will be a regular
> * mousedown/mousemove/click event."
>
> so I assume that comment is just bogus, since we certainly don't want to
> prevent mousedown/move/click events.
> Can't test that. Please do test.
True statement really, just a little misleading. When detected by the gesture module a pan gesture will get converted. But things that aren't considered panning (like finger taps) are handled independently. So you can click and pan with these changes. But you can't simulate a mouse down, mouse drag, and mouse up. We can live with that.
Assignee | ||
Comment 16•10 years ago
|
||
Assignee | ||
Comment 17•10 years ago
|
||
Attachment #8562057 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 18•10 years ago
|
||
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 19•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox38:
--- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 38
Updated•10 years ago
|
Iteration: --- → 38.3 - 23 Feb
Flags: qe-verify?
Updated•10 years ago
|
Flags: qe-verify? → qe-verify+
Comment 20•10 years ago
|
||
Verified using Nihglty 39.0a1 2015-03-09 on an Ultrabook with Win 8 64-bit.
Notes: - The scrolling is working on e10s windows, but it seems to be less smoothly than in regular window.
- Having a regular window and an e10s window side by side, scrolling in the regular window and then directly in the e10s window will cause both windows content to scroll. I will add a screencast if this is not already logged in bugzilla. This may not be related to e10s, I will investigate more tomorrow.
Comment 21•10 years ago
|
||
(In reply to Petruta Rasa [QA] [:petruta] from comment #20)
> Verified using Nightly 39.0a1 2015-03-09 on an Ultrabook with Win 8 64-bit.
>
> Notes: - The scrolling is working on e10s windows, but it seems to be less
> smoothly than in regular window.
Should I file a new bug for this?
> - Having a regular window and an e10s window side by side, scrolling in the
> regular window and then directly in the e10s window will cause both windows
> content to scroll. I will add a screencast if this is not already logged in
> bugzilla. This may not be related to e10s, I will investigate more tomorrow.
It seems that this issue reproduces back to Firefox 27.0.1, so it's not recent and not related to e10s. I will check to see if it's already logged.
Flags: needinfo?(jmathies)
Assignee | ||
Comment 22•10 years ago
|
||
(In reply to Petruta Rasa [QA] [:petruta] from comment #21)
> (In reply to Petruta Rasa [QA] [:petruta] from comment #20)
> > Verified using Nightly 39.0a1 2015-03-09 on an Ultrabook with Win 8 64-bit.
> >
> > Notes: - The scrolling is working on e10s windows, but it seems to be less
> > smoothly than in regular window.
>
> Should I file a new bug for this?
Sure. I noticed we don't async scroll have you remove your finger after a fling. That might be the bug the poster's referring to.
> > - Having a regular window and an e10s window side by side, scrolling in the
> > regular window and then directly in the e10s window will cause both windows
> > content to scroll. I will add a screencast if this is not already logged in
> > bugzilla. This may not be related to e10s, I will investigate more tomorrow.
>
> It seems that this issue reproduces back to Firefox 27.0.1, so it's not
> recent and not related to e10s. I will check to see if it's already logged.
Definitely needs filing.
Flags: needinfo?(jmathies)
Comment 23•10 years ago
|
||
Thanks Jim! I'm marking this issue as verified based on above comments.
Status: RESOLVED → VERIFIED
Comment 24•10 years ago
|
||
Version: 40.0a1(2015-04-09)
Scrollbars should always be draggable, but it didn't.
You need to log in
before you can comment on or make changes to this bug.
Description
•