Closed Bug 240363 Opened 20 years ago Closed 18 years ago

autoscroll should stop completely when window loses focus

Categories

(Firefox :: General, defect, P2)

defect

Tracking

()

RESOLVED FIXED
Firefox 3 alpha2

People

(Reporter: pantsgolem, Assigned: asaf)

References

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7b) Gecko/20040406 Firefox/0.8.0+
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7b) Gecko/20040406 Firefox/0.8.0+

Currently, if Firefox is autoscrolling when the window loses focus, autoscroll
does not stop.  While this is generally not an issue when the user changes the
focused window, it can cause problems when another window steals the focus.

Reproducible: Always
Steps to Reproduce:
1. Middle-click and mouse up/down to start autoscrolling (enable autoscroll if
it's not already enabled)
2. Switch to another window without clicking anywhere in Firefox
3. Switch back to Firefox again without clicking in the window itself
Actual Results:  
Firefox keeps happily scrolling along.

Expected Results:  
Autoscroll should stop as soon as the Firefox window loses focus.

I'm using KDE 3.2, in case it handles window focusing any differently than other
window managers.
See also bug 213496 which is somewhat related.
Autoscroll should probably stop working on any other event than mousemove, which
is basically the case in Internet Explorer
Confirmed.  Also occurs on an Alt+Tab task switch.
Blocks: 212273
Status: UNCONFIRMED → NEW
Ever confirmed: true
*** Bug 254806 has been marked as a duplicate of this bug. ***
*** Bug 295541 has been marked as a duplicate of this bug. ***
Assignee: firefox → bugs.mano
Severity: trivial → normal
OS: Linux → All
Priority: -- → P2
Hardware: PC → All
Target Milestone: --- → Firefox1.6-
Status: NEW → ASSIGNED
QA Contact: general
Attached patch patchSplinter Review
This actually fixes a few more cases (In particular, focusing something else in the chrome window or another document in this.contentWindow)
Attachment #245190 - Flags: review?(gavin.sharp)
Target Milestone: Firefox 2 → Firefox 3 alpha1
Attachment #245190 - Flags: review?(gavin.sharp) → review+
I will write a unit test once we have a the necessary framework in-place.
Flags: in-testsuite?
mozilla/toolkit/content/widgets/browser.xml 1.101
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Target Milestone: Firefox 3 alpha1 → Firefox 3 alpha2
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: