sambecker / exif-photo-blog

Photo blog, reporting 🤓 EXIF camera details (aperture, shutter speed, ISO) for each image.
https://photos.sambecker.com
711 stars 117 forks source link

Control the direction of the photos #20

Closed thjiang closed 9 months ago

thjiang commented 9 months ago

Hello, after uploading the photos, some of them have been rotated 90° in direction. May I ask how to control the direction of photos?

Here is an example: https://photos.thjiang.com/p/Kf4FK32A

sambecker commented 9 months ago

Very interesting. Have not encountered that issue yet. Can you post the original file so I can inspect?

thjiang commented 9 months ago

It seems that if the photo is taken vertically, the photo will be rotated. Here are two examples of the original image.

The exif orientation data for the second image is 8(Rotate 270 CW),maybe we need to check the orientation when displaying the photos ?

https://pic.ziyuan.wang/2023/11/27/guest_291f3d6895336.JPG https://pic.ziyuan.wang/2023/11/27/guest_9b8d8bf044350.JPG

sambecker commented 9 months ago

Just posted #22 to address this @thjiang—feedback welcome.

Will apply same thinking to aspectRatio before merging.

sambecker commented 9 months ago

Merged #22! Let me know if that worked @thjiang—will reopen if there are any other issues.

thjiang commented 9 months ago

Thank you very much for your prompt reply. I encountered some problems when deploying vercel so I couldn't verify it in time. I will check it as soon as possible after it is solved.

sambecker commented 9 months ago

encountered some problems when deploying vercel

Build errors? Let me know if any seem specific to this update!

thjiang commented 9 months ago

encountered some problems when deploying vercel

Build errors? Let me know if any seem specific to this update!

It's my own problem with using Vercel, not caused by this project

thjiang commented 9 months ago

Merged #22! Let me know if that worked @thjiang—will reopen if there are any other issues.

Many thanks for your help, it works perfectly now~

sambecker commented 9 months ago

Of course! So glad to hear.