Resizing to pixels should have a reasonable default value

Bug #513864 reported by Matt Perry
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Low
One Hundred Papercuts
Fix Released
Low
Unassigned
f-spot (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: f-spot

F-Spot has an "Export to Folder" feature which also allow resizing of the photo. If this option is enabled, the default value for resizing is zero. If the user accepts the default, F-Spot will give an error saying that the resized picture has zero width or height.

The resize value should have a sane default instead of defaulting to zero. I suggest defaulting to 400.

Tags: patch

Related branches

Changed in hundredpapercuts:
milestone: none → lucid-round-4
milestone: lucid-round-4 → lucid-round-7
status: New → Confirmed
Revision history for this message
Vish (vish) wrote :

Thank you for your bug report. This bug has been reported to the developers of the software. You can track it and make comments at:
https://bugzilla.gnome.org/show_bug.cgi?id=608440

Changed in f-spot (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Changed in hundredpapercuts:
importance: Undecided → Low
status: Confirmed → Triaged
Revision history for this message
yves (yves-kurz-gmail) wrote :
tags: added: patch
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work there. The upstream code seems to read some key value already, do you know why this one is 0? Wouldn't it be better to fix that value fetch rather?

Changed in f-spot (Ubuntu):
status: Triaged → Incomplete
Revision history for this message
Vish (vish) wrote :

This has now been fixed upstream.

Changed in f-spot (Ubuntu):
status: Incomplete → Fix Committed
Changed in hundredpapercuts:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot - 0.6.1.5-2ubuntu1

---------------
f-spot (0.6.1.5-2ubuntu1) lucid; urgency=low

  * debian/patches/git_copy_attribute_normal.patch:
    - git change to set the copy attributes to normal, fix a write permission
      issue (lp: #483268)
  * debian/patches/git_no_urlencode.patch:
    - git change to not urlencode the mail subjects (lp: #454972)
  * debian/patches/git_correct_description.patch:
    - git change to fix the profiles description displayed (lp: #502744)
  * debian/patches/git_color_profile_printing_breakage.patch:
    - git change to fix printing when a color profile is selected (lp: #496134)
  * debian/patches/git_key_conflict.patch:
    - git change to fix a shortcut conflict (lp: #516801)
  * debian/patches/git_set_default_resize_value.patch:
    - git change to set a correct default resize value (lp: #513864)
  * debian/patches/ubuntu_edit_tag.patch:
    - change the edit tag label to be consistent with other entries
      (lp: #513452)
  * debian/patches/ubuntu_transition_timing.patch:
    - change by Alex Launi to make the transition effect between modes take 1s
      (lp: #513045)
  * debian/patches/ubuntu_slider_animation_tweaking.patch:
    - change by Alex Launi to reduce the slider animation timing (lp: #513004)
  * debian/patches/ubuntu_check_duplicate.patch:
    - change by Matt Perry to use a better label for the duplicates checking
      (lp: #511767)
 -- Sebastien Bacher <email address hidden> Mon, 01 Mar 2010 16:06:41 +0100

Changed in f-spot (Ubuntu):
status: Fix Committed → Fix Released
Vish (vish)
Changed in hundredpapercuts:
status: Fix Committed → Fix Released
Changed in f-spot:
importance: Unknown → Low
status: Unknown → 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.