MIDI2LR icon indicating copy to clipboard operation
MIDI2LR copied to clipboard

Crop functions not considering photo orientation

Open onegrasshopper opened this issue 2 years ago • 1 comments

Describe the bug The current crop implementation seems to assume "AB" orientation photos only, not considering a photos getDevelopSettings()["orientation"] which could be "BC", "CD", or flipped, etc

This also affects the RatioCrop function in clientutilities.

Gives strange behavior on any photo not with AB orientation.

To Reproduce Steps to reproduce the behavior:

  • Load a few different orientation photos
  • Any photo that has orientation ~= 'AB' will behave strange in any of the crop functions

Expected behavior Consistent behavior

System(please complete the following information):

  • Lightroom version 11
  • MIDI2LR version 5.2

onegrasshopper avatar Jan 07 '22 08:01 onegrasshopper

As for now this cannot be fixed. The function returning the orientation (getDevelopSettings['Orientation']) seems to have a bug as it always returns 'nil'. So currently it is not possible to take the orientation into account within the plugin.

I reported a bug with Adobe for this.

wunditta avatar Sep 12 '23 11:09 wunditta