mysociety / fixmystreet

This is mySociety's popular map-based reporting platform: easy to install in new countries and regions
http://fixmystreet.org/
Other
515 stars 237 forks source link

Pixelated Logos on Android #282

Closed Jedidiah closed 11 years ago

Jedidiah commented 12 years ago

Congratulations on the new design. Just looking at the site on android and the logo on the header and the menu on the footer are very jaggy and pixelated. Looks like a much smaller image has been enlarged.

Android doesn't do screenshots on non-rooted phones and I don't have my camera with me so here are two very low quality "screenshots" from the webcam on my mac.

http://jedidiah.eu/img/fixmystreet-android2.jpg http://jedidiah.eu/img/fixmystreet-android.jpg

Footer looks a bit odd too.

dracos commented 12 years ago

Odd. Fine on my Android (2.3, Nexus S). Could you have a setting that's slightly zooming in somewhere? Even setting my default browser zoom to "Close", it just goes bigger and still not at all pixelated.

Jedidiah commented 12 years ago

It's a Samsung Galaxy S 2.2. I don't think it is an issue with the zooming, it's the same when I set the default zoom to "far" or "close" (currently on medium). It might be to do with holding such a large image sprite in memory on this older phone, everything else is nice and crisp.

struan commented 12 years ago

I'm seeing this on the emulator in the android phonegap app as well. I tried upping the memory there to 1024Mb and it didn't help.

Running it on a emulator with Ice Cream Sandwich does fix the problem though.

Jedidiah commented 12 years ago

Do you think it is connected to Webkit downsampling large images that people have been running into trying to make graphics for the retina iPad? "Images larger than 1775 pixels wide (or 1180 pixels high) get down-sampled by Webkit." unless they are progressive jpgs.

dracos commented 12 years ago

The site I found that quote on says "this issue only affects JPGs"? Anyway, our sprite.png could presumably be a lot smaller in pixel size regardless, which would at least test this.

abibroom commented 11 years ago

Transferred this issue to fixmystreet-mobile repo.