1266205 - Increase delay for scrollbar fadeout (or make it configurable for distributions)

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Android partner distribution, defect)

Description

[:Margaret Leibovic]

We receieved a partner request to increase our scrollbar fadeout time to 3 seconds. If we think this is too long, and something we don't want to do, we should explore making this configurable for distributions.

If we make a gecko pref for this, it could be easily set by distributions.

Comment 1

[Mike Kaply [:mkaply]]

To be clear this is the FADE_DELAY. As in how long we wait until the scrollbar goes away.

We are not allowing them to modify how long it takes to fade.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

With APZ enabled you can use the ui.scrollbarFadeBeginDelay pref to control this (defaults to 450ms in mobile.js).

Comment 3

[:Margaret Leibovic]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #2)
> With APZ enabled you can use the ui.scrollbarFadeBeginDelay pref to control
> this (defaults to 450ms in mobile.js).

In this case there may be nothing to fix here :)

Still waiting to hear back about timelines, hopefully this isn't urgent, in which case we can just wait for APZ to ride the trains.

Comment 4

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #2)
> With APZ enabled you can use the ui.scrollbarFadeBeginDelay pref to control
> this (defaults to 450ms in mobile.js).

So I take it ScrollbarLayer isn't used in APZ anymore? The last time I changed the constant, it was in that class:
  https://mxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/gfx/ScrollbarLayer.java#22

Comment 5

[Kartikaya Gupta (email:[e-mail])]

Correct, with APZ we use the gecko scrollbars and will be able to delete ScrollbarLayer entirely.

Comment 6

[Kartikaya Gupta (email:[e-mail])]

APZ is in release now, so the pref should be sufficient to change the fadeout delay.

Comment 7

[Mike Kaply [:mkaply]]

> APZ is in release now, so the pref should be sufficient to change the fadeout delay.

What's the pref exactly and how do I set? Thanks.

Comment 8

[Kartikaya Gupta (email:[e-mail])]

The pref is ui.scrollbarFadeBeginDelay which defaults to 450 in Fennec [1]. In theory you should be able to change it via about:config although I just tried that and about:config gives me up/down arrows for that pref which don't do anything and generally seems kinda busted. But you should still be able to change it by modifying mobile.js.

[1] http://searchfox.org/mozilla-central/source/mobile/android/app/mobile.js#61

Comment 9

[Mike Kaply [:mkaply]]

> In theory you should be able to change it via about:config although I just tried that and about:config gives me up/down arrows for that pref which don't do anything and generally seems kinda busted.

If you click on the number, you can edit it directly instead of using the arrows.

I've verified that this is customizable and works great.