cropping not working anymore

Bug #664317 reported by mypc
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Simple Scan
Fix Released
High
Unassigned
simple-scan (Ubuntu)
Fix Released
Medium
Unassigned
Maverick
Fix Released
Undecided
Unassigned

Bug Description

Impact: Saved images do not have crop as specified in GUI
Resolution: Upstream code change (patch in comments below)
To reproduce:
1. Open simple-scan
2. Scan a page
3. Click crop in toolbar
4. Adjust crop area to be small square in top right of image
5. Save document as JPEG
Expected result: Saved document is a small square
Observed result: Saved document is a tall rectangle

Patch has been tested and does not produce the above problem. Bug is well defined, is a logic error when the bottom of the crop was being incorrectly calculated

mypc (milisam250)
description: updated
description: updated
Changed in simple-scan:
status: New → Fix Committed
status: Fix Committed → Fix Released
affects: ubuntu → simple-scan (Ubuntu)
Changed in simple-scan:
importance: Undecided → High
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Patch that fixes problem

description: updated
tags: added: regression-release
Changed in simple-scan (Ubuntu):
status: New → Triaged
importance: Undecided → High
importance: High → Medium
status: Triaged → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted simple-scan into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in simple-scan (Ubuntu Maverick):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

Anyone could test the package in maverick-proposed and report here ?
Thanks in advance.

Revision history for this message
JP Penttilä (puurokauha-deactivatedaccount-deactivatedaccount) wrote :

I can confirm that cropping now works.

Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

Thanks for testing, that's much appreciated. Marking as verification-done.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package simple-scan - 2.32.0-0ubuntu4

---------------
simple-scan (2.32.0-0ubuntu4) maverick-proposed; urgency=low

  * debian/patches/04-crop-area.patch:
    - Fix crop area being incorrectly applied (LP: #664317)
  * debian/patches/05-brother3-grayscale.patch:
    - Fix text mode scanning not working using brother propietary driver (LP: #659536)
 -- Robert Ancell <email address hidden> Fri, 22 Oct 2010 15:00:30 +1100

Changed in simple-scan (Ubuntu Maverick):
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

copied to natty, too.

Changed in simple-scan (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
mypc (milisam250) wrote :

I confirm it works well now.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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