subsampling-scale-image-view icon indicating copy to clipboard operation
subsampling-scale-image-view copied to clipboard

support for the other 4 exif orientations

Open jkmcnk opened this issue 7 years ago • 3 comments

I added support for the yet unsupported exif orientations: flip {h,v}, trans{pose,verse}. thought I might as well contribute something back. so, here's the relevant code for your consideration.

you can test it with the exif orientation sample dataset here: http://k00.fr/kvfvy

best, j

jkmcnk avatar Jan 19 '18 10:01 jkmcnk

Looks interesting, did you do this because you needed the feature? Until now I don't think anyone had asked for it. It'll need some thorough testing which I don't have time for at the moment but I'll come back to it.

davemorrissey avatar Jan 23 '18 22:01 davemorrissey

yes. as far as testing goes, it was (and is being) tested with and without tiling, in a pan and zoom setting, in the android client for our storage service on a variety of devices, but I do suppose a more systematic approach to testing would be nice. ;) anyhow, glad to fix any issues that might arise.

jkmcnk avatar Jan 24 '18 10:01 jkmcnk

I don't plan to merge this but I'll leave this PR open so others can use this branch if they need this support.

davemorrissey avatar Feb 27 '20 00:02 davemorrissey