534015 - not enough of lower section of first-run page visible

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Madhava Enros [:madhava]]

The layout of the first-run page got changed a bit in the mm->px changes.  The lower section (starting with the section about weave) isn't as visible above the fold as it was before.

Here it is now: http://www.flickr.com/photos/madhava_work/4153358369/

Here it is as it was before: http://www.flickr.com/photos/madhava_work/4056534876/

We should have a bit more than is currently visible be visible (as in the second screenshot, above).  We can try dropping the font size, or dropping one "much" (Caitlin's looking into this).

Comment 1

[Madhava Enros [:madhava]]

Can't change the text as we're post freeze.  We could just widen the text width slightly to get rid of the third line.

Comment 2

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

Or reduce the fontsize by a pixel

Comment 3

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

Attachment: patch

Reduce font-size slightly to see more of the lower page.

With this patch I can see the "Get Up and Go" line as well as the first line of smaller text below it.

Comment 4

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

the patch makes the page match the old screenshot

Comment 5

[:Gavin Sharp [email: [e-mail]]]

Took the liberty of pushing this for you:
https://hg.mozilla.org/mobile-browser/rev/36623c53930a

Comment 6

[Aakash Desai [:aakashd]]

verified FIXED on builds:
Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2b5pre) Gecko/20091216 Firefox/3.6b5pre Fennec/1.0b6pre

and

Mozilla/5.0 (X11; U; Linux armv6l; Nokia N8xx; en-US; rv:1.9.3a1pre) Gecko/20091216 Firefox/3.7a1pre Fennec/1.0b5