The texture that is used on the background of the reader mode toolbar is much more pixilated than the one used on the area around the URL field at the top. This looks odd and they should use the same texture, in my opinion. See also bug 758804, which changed the texture of the upper toolbar to a more subtle version in the past.

I think Mark referenced this in channel, but I'm not sure if it was the bar or something else. I don't see it that much on my Galaxy Nexus.

Actually it look worse in reality than on the screenshot for me. I'm running a Galaxy S2 with Android 4.0.3 if that helps.

Might be because we don't provider toolbar background per screen density. Image is probably being scaled up. Ian, could please you provide the assets?

Yep, here you go Lucas.

Comment on attachment 644229 [details] [diff] [review] Properly set toolbar's background size to handle different densities I'm a little sad that we need different backgrounds for portrait and landscape, in addition to density. This makes me not want to use nosiy backgrounds. Too much waste.

I wonder if changing the way we apply the noise might help? Like just have one "noise" image that is a transparent overlay everywhere?

(In reply to Ian Barlow (:ibarlow) from comment #7) > I wonder if changing the way we apply the noise might help? Like just have > one "noise" image that is a transparent overlay everywhere? That could work. We'd still need one image per screen density though. But it's better than having 9 images.

Yeah. I think I'm going to open a separate bug about this, seems like a pretty worthwhile discussion to have, and not just within the scope of reader mode. :)

Pushed as we now have a follow up bug to work on optimizing out assets. http://hg.mozilla.org/integration/mozilla-inbound/rev/d9c31ba74029

Comment on attachment 644229 [details] [diff] [review] Properly set toolbar's background size to handle different densities [Approval Request Comment] Fix on reader mode which is planned for F16.

Pushed to Aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/3c61eb8b9a04