musselwhizzle / Focus-Points

Plugin for Lightroom to show which focus point was active in the camera when a photo was taken
Apache License 2.0
344 stars 42 forks source link

Dialog window too tall - 'Ok' button is hidden in some cases. #90

Closed philmoz closed 7 years ago

philmoz commented 7 years ago

The calculation of the photo size in the dialog window (FocusPointsDialog.calculatePhotoDimensions), should check if the photo size fits in the calculated content size.

Depending on the aspect ratio of the LR window compared to the image dimensions this can result in the dialog being larger than the screen height, and the 'OK' button is not visible.

Patch attached, this also increases the content size slightly to use more of the screen. windowsize.txt

musselwhizzle commented 7 years ago

Mind forking and PR? It's harder for me to merge it this way.

musselwhizzle commented 7 years ago

fixed