Closed Bug 1220199 Opened 4 years ago Closed 4 years ago

[B2GDroid] |scrollTo({behavior: 'smooth'...| doesn't work

Categories

(B2GDroid Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: etienne, Assigned: fabrice)

References

Details

Attachments

(1 file)

I've been told that APZ might only be working on the root scrollable area on Android, which will never work for Gaia since we have nested scrollable areas all the way down :)
Blocks: b2gdroid
Interestingly snapping is always instant.
We never smoothly scroll to the snapping point, once the snapping kicks in we instantly snap with no transition.
Looks like smooth scrolling in general doesn't work. Which points to the frames not being APZ enabled.
Are you building with --enable-android-apz in your mozconfig? I don't see that in the confvars for b2gdroid in-tree (but maybe I missed it). Maybe fabrice added that locally when he was building it?
(In reply to Etienne Segonzac (:etienne) from comment #2)
> Looks like smooth scrolling in general doesn't work. Which points to the
> frames not being APZ enabled.

This is still happening.
Pressing home to go back to the top of the homescreen, pressing home to close the utility tray etc...

All of these call |scrollTo({behavior: 'smooth'...| but we get no frame between the current position and the final position. It looks exactly like a "unsmooth" scrollTo().
Summary: [B2GDroid] Scrolling performance is so bad it feels like the pre-APZ days → [B2GDroid] |scrollTo({behavior: 'smooth'...| doesn't work
It's probably running with these two prefs set to false:

http://mxr.mozilla.org/mozilla-central/source/modules/libpref/init/all.js?rev=6111fe8aca39#3651

because of the #ifdef ANDROID. If you set them to true in b2gdroid.js or wrap them in a MOZ_ANDROID_APZ or something it should work.
Component: Panning and Zooming → General
Product: Core → B2GDroid
Works pretty well. Thanks Etienne for spotting that!
Assignee: nobody → fabrice
Attachment #8687333 - Flags: review?(bugmail.mozilla)
Comment on attachment 8687333 [details] [diff] [review]
smooth-scroll.patch

Review of attachment 8687333 [details] [diff] [review]:
-----------------------------------------------------------------

r+ for the scroll-behavior pref changes. I don't know about the raptor stuff...
Attachment #8687333 - Flags: review?(bugmail.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/683059d91038
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Duplicate of this bug: 1226271
Duplicate of this bug: 1202934
You need to log in before you can comment on or make changes to this bug.