Skip to content
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

Resizing an image changes it's orientation. #8310

Closed
Tamono opened this issue Jun 12, 2023 · 7 comments · Fixed by #8669
Closed

Resizing an image changes it's orientation. #8310

Tamono opened this issue Jun 12, 2023 · 7 comments · Fixed by #8669
Labels
bug It's a bug high High priority issues mobile All mobile platforms

Comments

@Tamono
Copy link

Tamono commented Jun 12, 2023

Description

When attaching a previously taken image to a note, Joplin changes its orientation.

Environment

Joplin version: 2.10.9
Platform: Android
OS specifics: Android 12

Steps to reproduce

  1. Take picture outside of Joplin
  2. Go into note
  3. Click on 3 dots. Select "Attach"
  4. Dialog pops up. Choose "Attach file"
  5. Select picture taken in step 1
  6. Resize dialog pops up. Select "Yes"
  7. Image has a wrong orientation.

Describe what you expected to happen

I expect the image to have it's original orientation after being resized by Joplin.

Logfile

log_file_screenshot

@Tamono Tamono added the bug It's a bug label Jun 12, 2023
@Tamono Tamono changed the title Resizing an image to 1920x1080 changes it's orientation. Resizing an image changes it's orientation. Jun 12, 2023
@laurent22
Copy link
Owner

Can you provide the image that's causing problem? In general it doesn't happen with most images as far as I know

@marph91
Copy link
Contributor

marph91 commented Jun 12, 2023

I had this issue a few weeks ago too, but can't reproduce it. I believe I rotated the image before by 90 degrees.

@MoriarT3a
Copy link

MoriarT3a commented Jul 11, 2023

I have this problem with every picture I shot in Portrait (I mean the picture is higher then wide).

The funny thing is, if I made some changes to the picture with another app, the orientation stays like it should be, after resizeing by Joplin.

I've made actually a picture here to reproduce it.

What happend:
Screenshot_20230711-211549_Trebuchet

Nd here is the Test Picture
IMG_20230711_211427165

I don't know if uploading the image maybe do something to my example image. Otherwiese I maybe could upload it to a cloud. 🤔

@github-actions
Copy link
Contributor

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? If you require support or are requesting an enhancement or feature then please create a topic on the Joplin forum. This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

@github-actions github-actions bot added the stale An issue that hasn't been active for a while... label Aug 11, 2023
@Tamono
Copy link
Author

Tamono commented Aug 12, 2023

I still have the same problem with every picture taken in portrait mode (meaning picture being higher than wide) as described above by @MoriarT3a. For me the image too keeps its orientation if I edit it outside of Joplin before attaching it. Pictures taken in landscape mode (meaning picture being wider than high) that are attached to a note keep their original orientation like the are supposed to.

@laurent22
Copy link
Owner

I don't know if uploading the image maybe do something to my example image. Otherwiese I maybe could upload it to a cloud. 🤔

I can't replicate the bug with your test image so indeed it may be modified by GitHub when you upload it. If you zip it up first and attach it that should preserve it

@MoriarT3a
Copy link

MoriarT3a commented Aug 12, 2023

I don't know if uploading the image maybe do something to my example image. Otherwiese I maybe could upload it to a cloud. 🤔

I can't replicate the bug with your test image so indeed it may be modified by GitHub when you upload it. If you zip it up first and attach it that should preserve it

@laurent22 Lets give it a try.

2go.zip

@Tamono are you using LineageOS? Maybe this is relatet to our OS or the Camera App.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug high High priority issues mobile All mobile platforms
Projects
None yet
4 participants