Image rotation does not work in Gpaint

Bug #262942 reported by Kevin Huang
4
Affects Status Importance Assigned to Milestone
gpaint (Debian)
Fix Released
Unknown
gpaint (Ubuntu)
Fix Released
Medium
Andrew Starr-Bochicchio
Hardy
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: gpaint

Scenario

1. draw a new paint
2. click "image", then "rotate right"---> image corrupted
3. click "image", then " rotate left" --> no effect

Revision history for this message
Caroline Ford (secretlondon) wrote :

Which version of gpaint is this?

On hardy neither rotate left nor rotate right seem to have any effect. I don't get a corrupted image.

Revision history for this message
Caroline Ford (secretlondon) wrote :

gpaint seems to have written this to my console:

** (gpaint:7055): WARNING **: TODO: cannot rotate if width is less than height.

Revision history for this message
Caroline Ford (secretlondon) wrote :

Creating a new image with width greater than height gives the same error.

Considering that width < height is the default this is really poor.

Revision history for this message
Caroline Ford (secretlondon) wrote :

Confirming that image rotation doesn't work.

Changed in gpaint:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Caroline Ford (secretlondon) wrote :

If I paste in a photograph left rotate seems to corrupt the image. It was drawings I made in gpaint that didn't seem to change.

Changed in gpaint:
status: Unknown → New
Revision history for this message
Goedson Teixeira Paixão (goedson) wrote :

Here's a patch to fix this issue. I'm including this in the (not uploaded yet) 0.3.3-2 revision of the package in Debian.
Complete source for the package can be found at http://git.debian.org/?p=users/goedson/pkg-gpaint.git;a=summary

Changed in gpaint:
status: New → Fix Committed
Changed in gpaint:
assignee: nobody → andrewsomething
status: Confirmed → In Progress
Changed in gpaint:
status: Fix Committed → Fix Released
Revision history for this message
Andrew Starr-Bochicchio (andrewsomething) wrote :

Sync bug #277725 has been filed.

Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into -proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gpaint - 0.3.3-2

---------------
gpaint (0.3.3-2) unstable; urgency=low

  * debian/patches/20_fix_line_width_combo.dpatch: Add missing line
    widths (5, 7 and 9).
  * debian/patches/10_fix_crash_on_font_select.dpatch: Make the font
    selection button active only when the text tool is selected,
    avoiding a crash that occurs if the user selects a font without
    selecting the text tool. (Closes: #497201) (LP: #262889)
  * debian/control: Added packaging Vcs information.
  * debian/patches/11_fix_image_rotation.dpatch: Use the
    gdk_pixbuf_rotate_simple function to do image rotation in multiples
    of 90 degrees (Closes: #497487) (LP: #262942)

 -- Andrew Starr-Bochicchio <email address hidden> Thu, 09 Oct 2008 07:42:09 +0100

Changed in gpaint:
status: In Progress → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Works fine for me now. Copied to hardy-updates.

Changed in gpaint:
status: New → 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.