539945 - portions of old page shown at bottom of new page when going from large page to smaller page

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: winmo screen shot

steps to reproduce:
1) load http://news.google.com
2) load http://mit.edu
3) pan down

you'll see some of the google news at the bottom of mit's home page

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: maemo screenshot

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: simple patch

Because of the orientation of the container, setting the height of tile-container below 500 was not possible.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Can you describe a little more why the <vbox> is needed? Couldn't we change the orientation of one of the existing containers?

Comment 4

[Aakash Desai [:aakashd]]

Is bug https://bugzilla.mozilla.org/show_bug.cgi?id=533491 related to this?

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #3)
> Can you describe a little more why the <vbox> is needed? Couldn't we change the
> orientation of one of the existing containers?

Nope, this was indeed my first idea but changing the orientation of the tile-container container to vertical, even if it resolved this bug, prevent horizontal scrolling.

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #4)
> Is bug https://bugzilla.mozilla.org/show_bug.cgi?id=533491 related to this?

I think not, this bug was mostly about small height website (or website very large compare to their height). 

On the screenshot of the bug i see that both of your page are supposed to fot the screen.

Comment 7

[Stuart Parmenter]

I don't think this should block 1.0 for maemo, unless we can come up with some worse examples.  The right fix will have to involve clearing tiles/removing them from the DOM when we change pages, which will lead to more checkerboarding most likely and will need to be carefully tested.