715928 - thumbnails not updated (or not able to be generated for some internal pages?)

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P2)

Description

[Dietrich Ayala (:dietrich)]

working from today's aurora build:

* thumbnail for about:home on about:home is blank white

* thumbnail for about:addons is light blue with the embossed firefox logo (the no-image image maybe?)

Comment 1

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

Ian, we purposely don't screenshot about:home because it'll just show up as all white. Do you want to include a stock thumbnail to show for it as a special case?

Comment 2

[Ian Barlow (:ibarlow)]

Hi Brad, we actually shouldn't be showing thumbnails of about: pages at all on the start page, as per bug 710392.

That said, as a general rule whenever thumbnails aren't showing up, we should be using the light blue placeholder image that Dietrich referred to.

Comment 3

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

Ian - what about in the tab menu? When the about:home page is open, it has a crappy tab thumbnail in the tab list (menu)

Comment 4

[Ian Barlow (:ibarlow)]

As per our conversation in IRC, I'll make a special "about:home" thumbnail for use in the tab menu, and post it here soon.

Comment 5

[Johnathan Nightingale [:johnath]]

Poke poke, Ian?

Comment 6

[Ian Barlow (:ibarlow)]

Attachment: Start Page placeholder thumbnails for tab menu

Comment 7

[Peter Retzer (:pretzer)]

(In reply to Ian Barlow (:ibarlow) from comment #6)
> Created attachment 600391 [details]
> Start Page placeholder thumbnails for tab menu

These thumbnails won't work out when about:home recieves a channel-branded styling (bug 710260). Is that considered a problem?

Comment 8

[Ian Barlow (:ibarlow)]

Hm, yes I would consider that a problem. 

I'll make some channel branded ones as well.

Comment 9

[Sriram Ramasubramanian [:sriram]]

I need new set of resources. Currently we use 136 X 78 for the thumbnails. @ibarlow: Could you please post newer ones?

Comment 10

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch

This patch shows the awesome thumbnail in Tabs-tray. :)
However, I need newer resources. Once I get them, I can replace those.

Comment 11

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

Comment on attachment 605950 [details] [diff] [review]
Patch


>+  res/drawable/abouthome.png \

>+  res/drawable-hdpi/abouthome.png \

>+  res/drawable-xhdpi-v11/abouthome.png \

>+            else if (TextUtils.equals(tab.getURL(), ABOUT_HOME))
>+                thumbnail.setImageResource(R.drawable.abouthome);

rename the resource to "abouthome_thumbnail"

Also, the resources have an extremely large footprint size. Can we try to reduce the size a bit?

r+ with the changes addressed. don't land until Ian get's newer (and hopefully smaller) resources.

Comment 12

[Ian Barlow (:ibarlow)]

Sriram, here are updated assets, also in Nightly and Aurora flavours! http://cl.ly/2x1b172x0w3M3K2y1y2X

(sorry for the cloud link, my bugzilla uploader is busted again)

I've included PNG and JPGs, feel free to use either. The PNGs are still around the same file size as before, but the JPGs are considerably smaller.

Comment 13

[Sriram Ramasubramanian [:sriram]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/f2f066e1abce

Pushed the patch for Firefox alone. Saving the other thumbnails to be landed when we implement content-branded UI.

Comment 14

[Mounir Lamouri (:mounir)]

https://hg.mozilla.org/mozilla-central/rev/f2f066e1abce

Comment 15

[Paul Feher]

Verified/fixed on:

Nightly Fennec 14.0a1 (2012-03-23)
Device: Samsung Nexus S
OS: Android 2.3.6

Comment 16

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

https://hg.mozilla.org/releases/mozilla-aurora/rev/ba444b71df2f