Skip to content
This repository has been archived by the owner on Jul 1, 2020. It is now read-only.

Scanner does not work in portrairt mode #336

Closed
dovidio opened this issue Jul 31, 2017 · 16 comments
Closed

Scanner does not work in portrairt mode #336

dovidio opened this issue Jul 31, 2017 · 16 comments

Comments

@dovidio
Copy link

dovidio commented Jul 31, 2017

First of all thanks for this amazing library, really made my android developer life easier :)
I have some problem with the last version of the app (1.9.5). I basically copied and pasted the SampleScanner example provided and the scanner works fine only in landscape mode, while in portrait mode the result handler is not even called. I have a galaxy s5 mini with Android 6.0 installed. Notice that I do not have the same problem if the dependency I add to gradle is 1.9.4.

@Komoi
Copy link

Komoi commented Aug 1, 2017

That happens on my side too, exactly as you described.
Android Studio 3 canary, OnePlus3T on 7.1.1

@dm77
Copy link
Owner

dm77 commented Aug 2, 2017

How annoying. The fix in 1.9.5 was supposed to resolve issues on Nexus 5x. Can you guys test the attached sample APK on your devices and see if you can still replicate the issue? Thank you so much for the help.

https://www.dropbox.com/s/v63vlzwsb4tl5ph/zxing-sample-debug.apk?dl=0

@Komoi
Copy link

Komoi commented Aug 2, 2017 via email

@dm77
Copy link
Owner

dm77 commented Aug 2, 2017

Thank you

@dm77
Copy link
Owner

dm77 commented Aug 2, 2017

I just went and bought a Samsung Galaxy Sol device hoping that I can replicate the bug, but unfortunately everything works fine.

@Komoi
Copy link

Komoi commented Aug 2, 2017 via email

@dm77
Copy link
Owner

dm77 commented Aug 2, 2017

Thank you @cyborg101 @Komoi for the feedback. I am going to revert the Nexus 5x fix for now and release 1.9.6 shortly.

@dm77 dm77 closed this as completed Aug 2, 2017
@dovidio
Copy link
Author

dovidio commented Aug 2, 2017

Thanks for the help @dm77, quite ironic that the two devices we use to debug at work are nexus 5x and a samsung galaxy s5 mini 👍

@dm77
Copy link
Owner

dm77 commented Aug 2, 2017

@cyborg101 Since you have both the problem devices, would you please test the attached APK file for me? That would be very helpful. I feel like the attached APK should fix portrait scanning on both devices. Really appreciate your help. https://www.dropbox.com/s/v63vlzwsb4tl5ph/zxing-sample-debug.apk?dl=0

@dm77 dm77 reopened this Aug 2, 2017
@Komoi
Copy link

Komoi commented Aug 2, 2017 via email

@dm77
Copy link
Owner

dm77 commented Aug 3, 2017

Thank you @Komoi

@dovidio
Copy link
Author

dovidio commented Aug 3, 2017

@dm77 I will try it tomorrow with the galaxy mini and will let you know

@dovidio
Copy link
Author

dovidio commented Aug 4, 2017

@dm77 Works in both orientations with galaxy s5 mini! Thanks again for your help!

@dm77
Copy link
Owner

dm77 commented Aug 4, 2017

Yay! Thanks for confirming @cyborg101

@Destiner
Copy link

Destiner commented Aug 9, 2017

Hmm. Looks like I am in minority here, but I couldn't get it worked on 1.9.7. I tried really hard, because I didn't know about the problem in previous versions. I tried 1.9.3 and it works instantly every time.
Running Nexus 5X with Android 7.1.2 (API 25).
It also works on 1.9.4 and fails on 1.9.5 and 1.9.6 :/

@androidovshchik
Copy link

i have noticed that autofocus bad works in portrait mode
although landscape works fine

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

No branches or pull requests

5 participants