"cmd_rotate" & "Change EXIF orientation ONLY"

Reported bugs that have been closed and/or resolved

Moderators: XnTriq, xnview, Dreamer

Post Reply
User avatar
XnTriq
Moderator & Librarian
Posts: 5366
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

"cmd_rotate" & "Change EXIF orientation ONLY"

Post by XnTriq » Mon Jul 11, 2016 1:30 am

Copied from “Question to various rotate”:
  1. Download Karl02's suite of test images (exif-orientation-testimages-v2.rar).
  2. Extract F2_5.jpg (EXIF orientation: 5 = left/top).
  3. Remove the write protection attribute from F2_5.jpg.
  4. Make a copy of F2_5.jpg.
  5. Rename F2_5.jpg to cmd_rotate90.jpg,
    and Copy of F2_5.jpg to cmd_lrotate90.jpg.
  6. Start XnViewMP.
  7. Make sure the following settings are activated:
    • General » File operations » Keep original date/time
      • Lossless transformations
    • Browser » Misc. » Rotation
      • Change EXIF orientation ONLY (if possible)
      • Use lossless rotation (if possible)
  8. Select the thumbnail of cmd_lrotate90.jpg in the browser and hit Ctrl+Shift+R (= cmd_lrotate90).
    1. Warning message: “JPEG lossless operation will modify original file(s), and may remove some unused pixels (width and height must be multiple of 8 pixels)! Do you want to continue?”
    2. The image is rotated 90° clockwise.
    3. The dimensions are changed from W255×H239 to W224×H255.
    4. The EXIF orientation flag is reset to 1 (top/left).
    5. The file's timestamp remains the same.
  9. Select the thumbnail of cmd_rotate90.jpg in the browser and click the Rotate clockwise (= cmd_rotate) button on the main toolbar.
    • No changes are made to the file :bug:
      Expected behavior: Steps 8a through 8e

User avatar
xnview
Author of XnView
Posts: 30902
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: "cmd_rotate" & "Change EXIF orientation ONLY"

Post by xnview » Thu Jul 14, 2016 3:20 pm

See Issue 918 for details.
Pierre.

User avatar
helmut
Posts: 8151
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: "cmd_rotate" & "Change EXIF orientation ONLY"

Post by helmut » Thu Feb 23, 2017 3:36 pm

This problem is supposed to be fixed in v0.82 or later. Could you please check and confirm, XnTriq?

User avatar
XnTriq
Moderator & Librarian
Posts: 5366
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: "cmd_rotate" & "Change EXIF orientation ONLY"

Post by XnTriq » Fri Feb 24, 2017 6:00 pm

Configuration:
  • ToolsSettings...BrowserMisc.Rotation
    • [×] Change EXIF orientation ONLY (if possible)
      [×] Use lossless rotation (if possible)
Test image:
Results:
  • cmd_rotate90 (Toolbar → Rotate clockwise) :bugfixed:
    cmd_rotate90.jpg
    cmd_rotate90.jpg (9.3 KiB) Viewed 324 times

    Code: Select all

                  {Width}: 255 pixels
                 {Height}: 239 pixels
       {EXIF:Orientation}: right-top (6)
                 {Size B}: 9'521 bytes
  • cmd_lrotate90 (ToolsJPEG lossless transformationsRotate clockwise) :bugfixed:
    cmd_lrotate90.jpg
    cmd_lrotate90.jpg (9.03 KiB) Viewed 324 times

    Code: Select all

                  {Width}: 240 pixels
                 {Height}: 239 pixels
       {EXIF:Orientation}: top-left (1)
                 {Size B}: 9'249 bytes

User avatar
XnTriq
Moderator & Librarian
Posts: 5366
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: "cmd_rotate" & "Change EXIF orientation ONLY"

Post by XnTriq » Fri Feb 24, 2017 10:15 pm

For comparison:
  • F2_1.jpg rotated 90° CW with Guido Vollbeding's Jpegcrop v2015.12
    jpegcrop.jpg
    jpegcrop.jpg (9.03 KiB) Viewed 318 times

    Code: Select all

                  {Width}: 240 pixels
                 {Height}: 239 pixels
       {EXIF:Orientation}: top-left (1)
                 {Size B}: 9'249 bytes
    [×] Transform Trim Edges (= Drop non-transformable edge blocks = Discard partial MCUs)
    p138995.png
    p138995.png (6.04 KiB) Viewed 318 times
Except for the embedded thumbnails (not automatically updated by Jpegcrop), cmd_lrotate90.jpg and jpegcrop.jpg are identical.

User avatar
helmut
Posts: 8151
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: "cmd_rotate" & "Change EXIF orientation ONLY"

Post by helmut » Sat Feb 25, 2017 4:36 pm

Thank you for your testing, XnTriq. The problem is fixed, right?

User avatar
XnTriq
Moderator & Librarian
Posts: 5366
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: "cmd_rotate" & "Change EXIF orientation ONLY"

Post by XnTriq » Sat Feb 25, 2017 5:30 pm

helmut wrote:Thank you for your testing, XnTriq. The problem is fixed, right?
Yes, confirmed fixed :bugfixed:
:arrow: Moved to “Closed/Resolved”

Post Reply