Closed
Bug 1316658
Opened 8 years ago
Closed 4 years ago
Bad horizontal swiping performance on https://www.cnet.com/tech-today/
Categories
(Firefox for Android Graveyard :: Toolbar, defect, P3)
Firefox for Android Graveyard
Toolbar
Tracking
(firefox49 affected, fennec+, firefox50 affected, firefox51 affected, firefox52 affected)
People
(Reporter: denschub, Unassigned)
References
(Blocks 1 open bug, )
Details
In this example, the horizontal swiping performance in Fennec is somewhat bad. On Desktop, transitioning between the slides is fast and fluent, but it's really slow and choppy on Fennec. As far as I can see, the site only does modify translateX() on the div inside div.contentParent.
The content slides are just divs laying next to each other, all visible and as far as I can see, nothing happens except for altering the translateX() property.
Updated•8 years ago
|
tracking-fennec: --- → ?
Comment 1•8 years ago
|
||
I was able to reproduce the issue using Nexus 5 (Android 6.0.1).
Setting the flags accordingly.
Updated•8 years ago
|
status-firefox49:
--- → affected
status-firefox50:
--- → affected
status-firefox51:
--- → affected
status-firefox52:
--- → affected
Version: unspecified → Trunk
Updated•8 years ago
|
Component: General → Graphics, Panning and Zooming
Comment 2•8 years ago
|
||
I'm not sure if there's much we can do here. The fact that the elements are large means that we can't pre-paint everything ahead of time and need to repaint during the animation/transition which makes it slow.
Blocks: paint-fast
Priority: -- → P3
Comment 3•8 years ago
|
||
(Speeding up paint, such as with the quantum render effort, is probably the best bet in terms of improving this performance)
tracking-fennec: ? → +
Comment 4•8 years ago
|
||
(In reply to away[Nov24,Dec5) Kartikaya Gupta (email:kats@mozilla.com) from comment #2)
> I'm not sure if there's much we can do here. The fact that the elements are
> large means that we can't pre-paint everything ahead of time and need to
> repaint during the animation/transition which makes it slow.
We can do better. In bug 1100357, we are implementing partial pre-rendering of large animated content (think rendering a "displayport" of the content rather than just the "viewport") so it can still be async-animated.
Depends on: 1100357
Comment 5•5 years ago
|
||
Unfortunately, the site now uses margin-left and left transitions instead of transform. :/
Comment 6•4 years ago
•
|
||
Dropping the dependency on bug 1100357 since the site no longer uses transform animations.
No longer depends on: 1100357
Comment 7•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•