751344 - focusing on the login textbox on m.facebook.com pans the site past where the page ends

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

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

Attachment: screenshot

STR:
1) load m.facebook.com
2) touch username field

ER: no horizontal panning (viewport meta tag should disallow it)
AR: noticeable white bar down the right side because we've panned past where the page ends

Comment 1

[Kartikaya Gupta (email:[e-mail])]

This seems to be a layout problem rather than a viewport issue. It's not that we pan the page, it's that the content actually shifts over to the left when we call scrollIntoView(false) on the text input.

I can reproduce this behaviour in desktop firefox by doing the following:

1. Install the UA switcher addon and add the Fennec 15 UA string and make it default
2. Resize the browser window to 360px wide (I think any width less than ~450px works for this)
3. Open the web console and run document.querySelector("input[type=text]").scrollIntoView(false)

Note how the page content gets shifted over and the white bar is visible down the right side, and the page is still not horizontally scrollable. Using Firebug to examine the layout properties of the various elements on the page shows no difference before and after step 3 above, so I'm not sure what's going on here. Widening the window and narrowing it again clears the white bar and allows doing step 3 again to reproduce.

Re-noming since I'm not sure this should be blocking for fennec; it should also be moved to core layout, maybe.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Attachment: Screenshot from desktop FF showing the same issue

Comment 3

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

Jet, can you hand this out?

Comment 4

[Jet Villegas (inactive)]

To Mats for a look...

Comment 5

[Mats Palmgren (inactive)]

Works for me, both in a current Nightly and the latest beta on Android.
(I did reproduce it a couple of weeks ago though, iirc)

Can anyone else still reproduce the problem?

Comment 6

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

works for me as well