gco / xee

Automatically exported from code.google.com/p/xee
74 stars 8 forks source link

Automatic zoom after rotation #269

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
If "shrink to fit" is selected, the zoom level should automatically be adapted 
after rotation of the image

What steps will reproduce the problem?
1. Select "shrink to fit" (automatic zoom)
2. Browse to a horizontal image
3. Rotate the image clockwise

Expected Result:
The image is rotated and the zoom level changes so that the full image is 
visible.

Current Result:
Image is rotated but the upper and lower part are clipped as the zoom level 
stays the same.

Version of Xee: 2.2
OSX 10.6.8

Apart from this amazingly nice and fast image viewer!

Original issue reported on code.google.com by flugsche...@gmail.com on 24 Aug 2011 at 1:44

GoogleCodeExporter commented 9 years ago
The problem here is that if the user has changed the zoom, it would be wrong to 
auto-zoom after rotation. Figuring out exactly what to do in any situation is a 
bit tricky.

Original comment by paracel...@gmail.com on 24 Aug 2011 at 2:26

GoogleCodeExporter commented 9 years ago
I understand your concern. However, I think this is probably the less common 
scenario. Usually, when browsing throw images (especially if they are fresh 
from the camera) I will want to rotate and have them zoom automatically again. 
If you think it might be a problem for some you could add a setting "Reset zoom 
after image modification" (or something like this). Or you could use the 
current behavior if "remember zoom level" is checked but not otherwise.

Also, can't you set a switch if the user zoomed an image after the auto-zoom? 
In this case you could leave the zoom level as it is after rotation. If the 
user didn't zoom manually, you could use the behavior I described.

I think this is how JustLooking behaves, which is actually a pretty nice viewer 
but just so slooooow compared to yours.

Original comment by flugsche...@gmail.com on 24 Aug 2011 at 7:39

GoogleCodeExporter commented 9 years ago
Xee 3 has different semantics for how to handle this, and should now handle it 
in a fairly acceptable fashion.

Original comment by paracel...@gmail.com on 23 Feb 2013 at 1:37