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 710479 - Default parameters for image resizing are misleading
Default parameters for image resizing are misleading
Status: RESOLVED FIXED
Product: gthumb
Classification: Other
Component: general
3.2.x
Other Linux
: Normal normal
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
Depends on:
Blocks:
 
 
Reported: 2013-10-18 21:35 UTC by Christian Krause
Modified: 2013-10-19 06:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
suggested patch to fix the problem (676 bytes, patch)
2013-10-18 21:35 UTC, Christian Krause
none Details | Review

Description Christian Krause 2013-10-18 21:35:22 UTC
Created attachment 257669 [details] [review]
suggested patch to fix the problem

The following bug report was filed for gthumb in Fedora:

https://bugzilla.redhat.com/show_bug.cgi?id=1006245

The user complaint that if he used the image resize dialog and tried to set the size to 800x800, gthumb would consume more and more memory and the operation would take very long.

When investigating the issue, I found out, that the default unit for resizing is "percent". On the other hand, the default values are 640x480. When just seeing the number, many users would expect that they are in pixels. The small drop-down list which shows "%" is not very prominent and easily overseen.

Especially in the use case, where the user wants to shrink digital photos and just enters a more appropriate size but not realizing that the unit is still "%", gthumb will look like that it hangs with a massive memory leak.

I suggest to change the default values to:
- either "%" with 100 as default values
- or "px" with 640x480

The attached patch will align data/org.gnome.gthumb.resize-images.gschema.xml.in
with org.gnome.gthumb.resize.gschema.xml.in ("%", with values 100, 100).
Comment 1 Paolo Bacchilega 2013-10-19 06:25:46 UTC
patch pushed now, thank you.