-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
galleria rotates a picture to landscape when switching to fullscreen #279
Comments
interestingly, update: i mean |
and obviously, i can't reproduce this now anymore. Not sure what was going on with that picture, but sorry for the noise. :/ |
Hmm weird that this happens only in fullscreen. The image rotation stuff is confusing, but as long as Pillow does not allow to modify the exif after rotation (#72) there is no good solution. |
yeah this is just totally weird, especially since i can't reproduce anymore. i wonder if there could have been some caching issues... i'll bump this again if i have the problem again later. |
oh and just another thought i had... one thing with image rotation is that browsers do know how to rotate images on their own. a portrait photo will get displayed vertically if opened directly (ie. not inlined in a |
okay well, this damn bug is back. and now i have public photos to show it, which might make things easier here... https://photos.anarc.at/cities/montreal/#/3 Note that this photos is fine: https://photos.anarc.at/cities/montreal/#/4 The difference? The latter was exported by Darktable, while the former is straight from the camera... Could this be why there is a problem here? That pillow wouldn't detect the proper exif tags from the camera? |
I see both with the correct (vertical) orientation |
even in full screen? |
here's a screencast (zip file because github doesn't support video!) of the problem from my point of view, just to make sure we're on the same page. this is |
Ok, I can also see this actually, but I need to wait long enough because my network is slow and the files are big and galleria shows the low res file while it downloads the high res! So maybe an issue with the fullscreen plugin ? It would be interesting to check in galleria's changelog or git repo is there is something. I think there is more recent versions than the one bundled in sigal, but last time I tried to update the design was broken. |
yeah, i was wondering if this could be solved by a theme update as well... i can't imagine that galleria has such a huge bug and no one noticed (although I did look at their issue queue, iirc and couldn't find anything)... |
my strategy right now is to wait for theme updates in sigal and try again :) |
You're lucky, I pushed a branch with the theme update yesterday, and I just merged it (#296) :) |
nice, i guess i need to try this out now! |
at first glance, this seems fixed in 1.4.0 - thanks!!! |
Good news! :) |
hmm... unfortunately, it seems I am still seeing this bug in 1.4.0, sorry for the false positive. :( it does not seem like Galleria 1.5.7 fixed the problem and i still see originals rotate all over the place... |
There's something really odd with a picture I have. Some other pictures of mine don't have that problem, but for some reason, when I switch to fullscreen in galleria with this special picture, it gets flipped as landscape even though it was displayed, properly, in portrait mode, when in normal mode.
Worse: when I go back from fullscreen, the image is still squeeze to the side, like this:
It's as if something inside galleria was flipping the image. This happens only when I have
keep_orig
enabled, here's the (minimal) config to reproduce this:I do not have consent to publish the image, so I would prefer if I could send you a sample in private.
Other themes do not seem to have that problem. Setting
keep_orig
toFalse
works around the issue as well. I can reproduce this in Firefox 57 and Chromium 63.0.3239.84.The text was updated successfully, but these errors were encountered: