Closed Bug 509450 Opened 11 years ago Closed 11 years ago

When a new page begins loading, we should move to the top of the page

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(fennec1.0b3+)

VERIFIED FIXED
Tracking Status
fennec 1.0b3+ ---

People

(Reporter: pavlov, Assigned: pavlov)

References

Details

Attachments

(1 file)

I believe the right place to do this is in onLocationChange. We may also want to look at adding a nowreallyloading call on tab, that kicks off the resizeAndZoom code rather than in startLoading
Attached patch fixSplinter Review
Assignee: nobody → pavlov
Attachment #394117 - Flags: review?(froystig)
Attachment #394117 - Attachment is patch: true
Attachment #394117 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 394117 [details] [diff] [review]
fix

Looks good to me.  

Small nits to fix: I'd like to see the other calls to bv.naftervisiblemove fixed to remove the parameters, and I'd like to see the value of getElementById("tile-container") cached in the browser object.
Attachment #394117 - Flags: review?(froystig) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
verified with nightly build from 08/13/2009
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.