212273 - [META] Use "better" Autoscroll for Firefox

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Marc Boullet]

Some people think that the autoscroll version in AiO is "better" and should be
the basis for the "standard" one in FireBird. See
http://forums.mozillazine.org/viewtopic.php?t=14971&postdays=0&postorder=asc&start=10

As the author if AiO Autoscroll enhancements, I ported the AiO autoscroll in
browser.xml. See attachment in next post.

Enhancements:
  - smooth scrolling speed changes (so called pseudo-logarithmic).
  - better "something to scroll" detection: ability to scroll in divs with
overflow: auto, textareas, etc.

Comment 1

[Marc Boullet]

Attachment: Patch against 07-06 browser.xml

Comment 2

[Steffen Wilberg]

AiO = All-in-One Extension:
http://perso.wanadoo.fr/marc.boullet/ext/extensions-en.html

Comment 3

[Simon Paquet [:sipaq]]

Comment on attachment 127429 [details] [diff] [review]
Patch against 07-06 browser.xml

Blake, can you take a look at this?

Comment 4

[Marc Boullet]

Attachment: New patch with fix to bug #211858

Comment 5

[Mike Connor [:mconnor]]

--> enhancement

Comment 6

[Steffen Wilberg]

This also fixes bug 212072.
I think this bug is more than just an enhancement because it fixes two other bugs.

Comment 7

[noririty]

Steffen:
That's right.

-> OS: Windows
Tweaking summary to be a meta bug.

Comment 8

[Steffen Wilberg]

The patch is bit-rotted because Blake modified browser.xml twice since then. I
wonder why he's fixing the bugs of the native autoscroll instead of just
switching to the AiO one.

Comment 9

[noririty]

We are still investigating to switch AiO. Please wait for a while. Thanks.

Comment 10

[Steffen Wilberg]

checked in by noririty between 03:34 and 04:21 today.

Comment 11

[amano]

but why isn't it marked FIXED then?

Comment 12

[Steffen Wilberg]

Because it's a meta bug. Fixing this depends on all the dependant bugs being fixed.

Comment 13

[noririty]

backing out AiO because of performance and some event handling issues.

Comment 14

[Jens Bannmann]

When/if AiO's implementation gets ported, we should make sure that bug 213250 is
fixed in that extension's code, too. I first noticed it in the extension, and
found it again in a recent nightly. See my comment at
http://bugzilla.mozilla.org/show_bug.cgi?id=213250#c7 on how to fix it (for the
current non-AiO impl).

Comment 15

[David P James]

We also have to do something about the conflicting nature of autoscroll and
contentLoadURL in Linux, particularly since both are enabled by default. Right
now it's simply a usability nightmare as they fight each other.

In bug 216899 I suggest a solution that will allow both features to co-exist and
not conflict. How difficult this will be to code I'm not sure, but it does
involve giving precedence to contentLoadURL rather than autoscroll (autoscroll
currently seems to have precedence in most cases).

Comment 16

[Robert Parenton]

Bug 242621 has been filed for moving the autoscroll icon out of the DOM, which
would solve at least five of the above bugs.

Comment 17

[fyodor]

Robert, would this include not being able to scroll with wheel or autoscroll AT
ALL in e.g. DIVs with overflow=scroll? Or is this a separate bug?

Comment 18

[Robin Monks]

*** Bug 291134 has been marked as a duplicate of this bug. ***

Comment 19

[k0255886]

Attachment: Sorry

Test it

Comment 20

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

With bug 242621 fixed, we have a "better" autoscroll.  A couple deps for this bug are still open, but it might be ok to resolve this metabug.

Comment 21

[Mark Banner (:standard8)]

The dependencies of this are all fixed and hasn't been used in ages.