search

darickc / MMM-BackgroundSlideshow

MIT License
138 stars 79 forks source link

Handle all values of EXIF orientation #55

Closed playmobil42 closed 4 years ago

playmobil42 commented 4 years ago

Previously only values of 6 & 8 were handled.

Testing: tested manually with images from https://github.com/recurser/exif-orientation-examples.

This CL is not perfect, as CSS transforms also apply to the containing div's dimensions esp. flipping width/height on rotation, but it's better than the previous state.

playmobil42 commented 4 years ago

This is a step in the right direction to address #46

jritsema commented 2 years ago

Thanks @playmobil42! This improves things for me.