konklone / congress-android

Congress for Android, an app for tracking Congress.
https://play.google.com/store/apps/details?id=com.sunlightlabs.android.congress
Other
157 stars 42 forks source link

Port the Image loader to Picasso #635

Closed paultag closed 10 years ago

paultag commented 10 years ago

Depends on #634 and #633 (for no reason other than ordering)

konklone commented 10 years ago

Images and memory management are one of the more annoying sources of bugs in the app, so definitely give this some battle testing before releasing to production.

Also, adding Picasso means that we now depend on OkHttp for other reasons than Mapbox' SDK. Maybe add a note in the comments to the top of https://github.com/sunlightlabs/congress-android/blob/master/src/com/sunlightlabs/android/congress/utils/HttpManager.java explicitly listing our two dependencies on OkHttp.