After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 657977 - Allow cropping of image when picking avatar
Allow cropping of image when picking avatar
Status: RESOLVED FIXED
Product: gnome-contacts
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Contacts maintainer(s)
GNOME Contacts maintainer(s)
Depends on:
Blocks: 657978
 
 
Reported: 2011-09-01 17:57 UTC by Alexander Larsson
Modified: 2012-02-07 14:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Adding support for UmCropArea widget (32.81 KB, patch)
2012-02-06 15:33 UTC, Erick Perez Castellanos
none Details | Review
Adding support for avatar cropping (6.24 KB, patch)
2012-02-06 15:36 UTC, Erick Perez Castellanos
none Details | Review

Description Alexander Larsson 2011-09-01 17:57:49 UTC
We need to support cropping of the avatar by selecting a rectangular area when picking an avatar from file. control-center has some widgetry that does this we could reuse. Control center does the cropping in a separate dialog, but I think it would look better if we do the cropping inside the file chooser itself.

This depends on bug 657976 so we can push the cropped image into folks.
Comment 1 Erick Perez Castellanos 2012-02-06 15:33:43 UTC
Created attachment 206902 [details] [review]
Adding support for UmCropArea widget
Comment 2 Erick Perez Castellanos 2012-02-06 15:36:56 UTC
Created attachment 206903 [details] [review]
Adding support for avatar cropping
Comment 3 Erick Perez Castellanos 2012-02-06 15:41:15 UTC
These are the patch for fixing this.
Though today I realized that maybe it gets in the way of the development of other stuffs.
I tried to make it not a separate dialog, cause I agree it would look awful, though, now I think is somewhat poorly discoverable.
Comment 4 Alexander Larsson 2012-02-07 14:31:31 UTC
Looks good, pushed to master.