Twitux deals badly with missized images

Bug #339257 reported by Aquarion
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
twitux (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: twitux

Very occasionally when you upload a new avatar to Twitter, they fail to resize it before sending it on to the Amazon servers where the avatars are hosted. In one case on my twitter feed someone has managed to get a 800*600 image though.

It is therefore necessary for Twitux to scale down any avatar pictures it loads into a form that will not completely destroy the layout of the application, in case Twitter have broken something again.

The person on my list who has broken it is http://twitter.com/botherer, his avatar image is currently http://s3.amazonaws.com/twitter_production/profile_images/76834457/imustkillalloftherobots_bigger.gif

An option to turn off avatars would be good as well.

Revision history for this message
Huntly Cameron (huntly-cameron) wrote :

I've had that problem as well. Something to resize the avatar before being displayed would be nice. Or as stated above the option to turn them off.

Revision history for this message
justpaul (justpaul) wrote :
Revision history for this message
Pobice (robert-pobice) wrote :

I've just run into the same problem with Jamesallenonf1

Changed in twitux (Ubuntu):
status: New → Confirmed
Revision history for this message
Pobice (robert-pobice) wrote :
Revision history for this message
Antoine Leclair (antoineleclair) wrote :

I'm not familiar with how to contribute to GNOME / Twitux, but I found how to fix it.

In the source code, in src/twitux-app.c, change line 1473 from:
 pixbuf = gdk_pixbuf_new_from_file (file, &error);
to:
 pixbuf = gdk_pixbuf_new_from_file_at_size (file, 48, 48, &error);

That way, the picture is left untouched in the cache folder ($HOME/.gnome2/twitux/avatar) but is scaled to fit the 48x48px size when loaded, keeping the original aspect ratio.

If I find time to do so, I'll try to submit a patch to the projet.

Revision history for this message
Antoine Leclair (antoineleclair) wrote :

It seems to have already been corrected in the SVN of the project on sourceforge.

Revision history for this message
Mark East (feasty) wrote :

I have added a patch for this issue (it also fixes bug 412390 in the same patch). The fix is as described by Antoine Leclair.

Revision history for this message
Stephan Rügamer (sruegamer) wrote :

working on it

Changed in twitux (Ubuntu):
assignee: nobody → Stephan Hermann (shermann)
status: Confirmed → In Progress
Revision history for this message
Stephan Rügamer (sruegamer) wrote :

twitux (0.69-0ubuntu2) lucid; urgency=low

  * Fixed bug with avatar image resizing.
    - Bug #339257
  * Fixed issue with avatars using the same name overwriting one another.
    - Bug #412390

Date: Fri, 05 Feb 2010 20:10:13 +0000
Changed-By: Mark East <email address hidden>
Maintainer: Ubuntu Developers <email address hidden>
Signed-By: Stephan Hermann <email address hidden>
https://launchpad.net/ubuntu/lucid/+source/twitux/0.69-0ubuntu2

Changed in twitux (Ubuntu):
assignee: Stephan Hermann (shermann) → nobody
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.