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

Android 13, Titanium 12 update #27

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

m1ga
Copy link

@m1ga m1ga commented Jan 19, 2023

Basically the same as #26 but with an update to Ti 12 (minsdk!) using READ_MEDIA_IMAGES check for Android 13

@cmtanko
Copy link

cmtanko commented Aug 27, 2023

@m1ga I was trying this new version, and everything seemed to be fine but then realised there is an app crash (just the first time when it asks for permission, successive imagepicker works great).

Replication Step:

  • Reinstall the app
  • Go to the Photo Gallery (Select Photos)
  • It loads a blank white screen, waits for a few seconds, and then gets App isn'tnt responding, Do you want to wait or close app.

Last working version was: 3.0.0

@m1ga
Copy link
Author

m1ga commented Aug 28, 2023

Reinstall the app

The example app from the repo? I've updated it so it will ask for permissions too (and fixed a typo) and it works fine on my Pixel 7 (Android 13).

Also removed those toColor deprecation warnings. The ANR questions sounds more like a memory issue. Did you select multiple images that you try to resize in the app?

@cmtanko
Copy link

cmtanko commented Aug 28, 2023

I'm stuck in a page where it says "Select Photos" , (but nothing loads except for headers) and then after few seconds get
"App isn't responding" [Close app] [Wait] . So, I am not able to select anything anyways. Let me try on a clean sample project as well.

Device: Samsung S20, Android:13

@Chris0714
Copy link

@m1ga @cmtanko Okay, I found the reason. Don't close the index.js window, I mean the first open window, otherwise the app will crash when opening the gallery.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants