Page 1 of 1

"cmd_rotate" & "Change EXIF orientation ONLY"

Posted: Mon Jul 11, 2016 1:30 am
by XnTriq
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

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

Posted: Thu Jul 14, 2016 3:20 pm
by xnview
See Issue 918 for details.

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

Posted: Thu Feb 23, 2017 3:36 pm
by helmut
This problem is supposed to be fixed in v0.82 or later. Could you please check and confirm, XnTriq?

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

Posted: Fri Feb 24, 2017 6:00 pm
by XnTriq
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 1148 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 1148 times

    Code: Select all

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

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

Posted: Fri Feb 24, 2017 10:15 pm
by XnTriq
For comparison:
  • F2_1.jpg rotated 90° CW with Guido Vollbeding's Jpegcrop v2015.12
    jpegcrop.jpg
    jpegcrop.jpg (9.03 KiB) Viewed 1142 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 1142 times
Except for the embedded thumbnails (not automatically updated by Jpegcrop), cmd_lrotate90.jpg and jpegcrop.jpg are identical.

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

Posted: Sat Feb 25, 2017 4:36 pm
by helmut
Thank you for your testing, XnTriq. The problem is fixed, right?

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

Posted: Sat Feb 25, 2017 5:30 pm
by XnTriq
helmut wrote:Thank you for your testing, XnTriq. The problem is fixed, right?
Yes, confirmed fixed :bugfixed:
:arrow: Moved to “Closed/Resolved”