198992 - Set general.smoothScroll to true by default in all.js

Status: VERIFIED FIXED

(Firefox :: Settings UI, enhancement)

Description

[Sébastien Delahaye]

Since bug 174049 got fixed, smooth scrolling is now built into Phoenix, but not
enabled by default.
We should set general.smoothScroll to true by default in all.js.

No UI is needed for this.

There is a similar bug for Mozilla: bug 198964.

Comment 1

[David Tenser [:djst]]

Reassigning.

Comment 2

[Sébastien Delahaye]

Attachment: patch

Comment 3

[David Tenser [:djst]]

Comment on attachment 118306 [details] [diff] [review]
patch

Looks good.

Comment 4

[Stefan Borggraefe]

IMHO this should be held back until Bug 199107 and Bug 199024 are fixed.

Comment 5

[lohphat]

After installing 1.4a, the pref was not added, I had to do it manually.  Should
it have been?

Comment 6

[Jacek Piskozub]

lohphat: The answer is yes. Please, read the release notes (in this case "What's
New in Mozilla 1.4a" section).

Comment 7

[lohphat]

You may use the about:cpnfig interface frequently, but I do not. It's
non-obvious that the interface can be used to add prefs.  Ijust headed to the
prefs.js file directly.

Comment 8

[lohphat]

Related question:  Shouldn't pop-up contect menus replicate actions available
from the pull-downs?  The right-clock actions of adding prefs in the
about:config window are not found in the edit pulldown menu.

Comment 9

[Pierre Chanial]

oopss, I haven't seen the patch.
Anyway, marking fixed.

Comment 10

[noririty]

v.

Comment 11

[Sébastien Delahaye]

Since it's fixed, marking 0.6.